From 952aae108db972a19d0ed9f6a98b68dc4c12b963 Mon Sep 17 00:00:00 2001 From: Douglas Coburn Date: Wed, 19 Feb 2025 19:48:28 -0800 Subject: [PATCH 1/3] Fix for params already being a dict --- socketdev/repos/__init__.py | 4 ++-- socketdev/version.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/socketdev/repos/__init__.py b/socketdev/repos/__init__.py index fb4d50a..9f70aec 100644 --- a/socketdev/repos/__init__.py +++ b/socketdev/repos/__init__.py @@ -131,7 +131,7 @@ def post(self, org_slug: str, **kwargs) -> dict: return {} path = "orgs/" + org_slug + "/repos" - response = self.api.do_request(path=path, method="POST", payload=params.__dict__) + response = self.api.do_request(path=path, method="POST", payload=params) if response.status_code == 200: result = response.json() @@ -150,7 +150,7 @@ def update(self, org_slug: str, repo_name: str, **kwargs) -> dict: return {} path = f"orgs/{org_slug}/repos/{repo_name}" - response = self.api.do_request(path=path, method="POST", payload=params.__dict__) + response = self.api.do_request(path=path, method="POST", payload=params) if response.status_code == 200: result = response.json() diff --git a/socketdev/version.py b/socketdev/version.py index b0747c8..f30c4f0 100644 --- a/socketdev/version.py +++ b/socketdev/version.py @@ -1 +1 @@ -__version__ = "2.0.5" \ No newline at end of file +__version__ = "2.0.6" \ No newline at end of file From 36212a19143683225b088fe751813163b7224c27 Mon Sep 17 00:00:00 2001 From: Douglas Coburn Date: Wed, 19 Feb 2025 19:55:05 -0800 Subject: [PATCH 2/3] Payload needed to be str not dict --- socketdev/repos/__init__.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/socketdev/repos/__init__.py b/socketdev/repos/__init__.py index 9f70aec..6f18de0 100644 --- a/socketdev/repos/__init__.py +++ b/socketdev/repos/__init__.py @@ -1,3 +1,4 @@ +import json import logging from typing import List, Optional from dataclasses import dataclass, asdict @@ -131,7 +132,8 @@ def post(self, org_slug: str, **kwargs) -> dict: return {} path = "orgs/" + org_slug + "/repos" - response = self.api.do_request(path=path, method="POST", payload=params) + payload = json.dumps(params) + response = self.api.do_request(path=path, method="POST", payload=payload) if response.status_code == 200: result = response.json() @@ -150,7 +152,8 @@ def update(self, org_slug: str, repo_name: str, **kwargs) -> dict: return {} path = f"orgs/{org_slug}/repos/{repo_name}" - response = self.api.do_request(path=path, method="POST", payload=params) + payload = json.dumps(params) + response = self.api.do_request(path=path, method="POST", payload=payload) if response.status_code == 200: result = response.json() From c671c767715d9e56a44cddbe8fbf8903c54bd0a1 Mon Sep 17 00:00:00 2001 From: Douglas Coburn Date: Wed, 19 Feb 2025 19:58:35 -0800 Subject: [PATCH 3/3] Fixed check for the right status code --- socketdev/repos/__init__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/socketdev/repos/__init__.py b/socketdev/repos/__init__.py index 6f18de0..8328abd 100644 --- a/socketdev/repos/__init__.py +++ b/socketdev/repos/__init__.py @@ -135,7 +135,7 @@ def post(self, org_slug: str, **kwargs) -> dict: payload = json.dumps(params) response = self.api.do_request(path=path, method="POST", payload=payload) - if response.status_code == 200: + if response.status_code == 201: result = response.json() return result