Skip to content

Commit 19cdeee

Browse files
Koushik SKclaude
andcommitted
fix: correct Requirements params and ensure multipart for create/update
- Requirements.get() now only sends non-None params instead of always sending all three (which resulted in ?country_iso=None queries) - _build_compliance_multipart returns empty dict instead of None when no documents, ensuring create/update always use multipart/form-data Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 4a507c8 commit 19cdeee

1 file changed

Lines changed: 8 additions & 3 deletions

File tree

plivo/resources/phone_number_compliance.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,17 @@ class PhoneNumberComplianceRequirements(PlivoResourceInterface):
1313

1414
def get(self, country_iso=None, number_type=None, user_type=None):
1515
# GET /PhoneNumber/Compliance/Requirements
16+
params = {}
17+
if country_iso:
18+
params['country_iso'] = country_iso
19+
if number_type:
20+
params['number_type'] = number_type
21+
if user_type:
22+
params['user_type'] = user_type
1623
return self.client.request(
1724
'GET',
1825
('PhoneNumber', 'Compliance', 'Requirements'),
19-
dict(country_iso=country_iso, number_type=number_type, user_type=user_type)
26+
params
2027
)
2128

2229

@@ -114,6 +121,4 @@ def _build_compliance_multipart(data, documents):
114121
for idx, doc_path in enumerate(documents):
115122
field_name = 'documents[{}].file'.format(idx)
116123
files[field_name] = (os.path.basename(doc_path), open(doc_path, 'rb'))
117-
if not files:
118-
files = None
119124
return payload, files

0 commit comments

Comments
 (0)