[Python] REST API request vraag

[ironheart]

Mijn vraag
Ik vermoed dat dit het juiste topic is dus hierbij mijn vraag;
Ik heb een REST API geformuleerd in Python welke met TransIP dient te kletsen en op die manier een nieuw DNS record aanmaakt. Voordat ik deze in productie zet wil ik graag weten of dit de correcte wijze en of jullie wellicht verbeteringen hebben.

Relevante software en hardware die ik gebruik
Ik heb de TransIP API documentatie erop nageslagen en alles lijkt me ok geformuleerd (nog niet getest).

Wat ik al gevonden of geprobeerd heb
Mijn code:

code:

import requests
import json

url = "https://api.transip.nl/v6/domains/DOMEIN/dns"  

payload = {
    "name": "subdomain1",
    "expire": "3600"
    "type": "A"
    "content":"92.92.92.92"
}

json_payload = json.dumps(payload)


headers = {
    "Content-Type": "application/json",
    "Authorization": "Bearer -----BEGIN PRIVATE KEY-----XXXXXXXXXXXXX-----END PRIVATE KEY-----"}

response = requests.post(url, data=json_payload, headers=headers)

if response.status_code == 200:
    print("POST request was successful.")
else:
    print("POST request failed with status code:", response.status_code)

print("Response:", response.text)

[ Voor 2% gewijzigd door RobIII op 10-06-2023 00:15 . Reden: Private key verwijderd ]

[Hero of Time]

Klinkt meer als een stukje softwareontwikkeling, dan een vraag over een client OS. Want Python kan je net zo goed op Windows en MacOS gebruiken en is derhalve niet specifiek een Linux iets.

LNX -> DEV.

[erwn]

@ironheart , als dat een echte key is in de authorisation header, zou ik 'm revoken en een nieuwe genereren.

[RobIII]

@ironheart Wat @erwn zegt: Die private key heet niet voor niets private key. Die hoor je voor jezelf te houden en is nu dus gecompromitteerd. Je zult een nieuw keypair moeten genereren en de huidige key revoken.

Verder zie ik geen problemen in je code; ik denk dat het handig(er) is als je een concrete vraag stelt over datgeen waar je twijfels over hebt. Ik ben geen python expert maar ik zie hier niets verkeerds en 't ziet er vrij logisch uit.

[ironheart]

Ok, ik ga de code testen. Dank allemaal maar de private key was uiteraard niet de echte private key.

[Ben(V)]

Volgens mij is je api aanroep naar Trasnip niet correct.
Jij gebruikt:

url = "https://api.transip.nl/v6/domains/DOMEIN/dns"

Volgens mij moet het zijn:

url = "https://api.transip.nl/v6/domains/"

Zie:
https://api.transip.nl/re...3391-674753614.1686383391

En de returncode is ook geen 200 maar 201 als het goed gaat.

[Merethil]

Ben(V) schreef op zaterdag 10 juni 2023 @ 10:06:
Volgens mij is je api aanroep naar Trasnip niet correct.
Jij gebruikt:

[...]

Volgens mij moet het zijn:

[...]


Zie:
https://api.transip.nl/re...3391-674753614.1686383391

En de returncode is ook geen 200 maar 201 als het goed gaat.

Zijn oorspronkelijke URL is toch echt het endpoint voor toevoegen van een dns record:
https://api.transip.nl/re...86383391#domains-dns-post

Response is wel 201 idd. Ik zou 2.* als check op de responsecode gebruiken om te zien of het goed ging; dan dek je 200 en 204 ook (want wie weet is de documentatie brak )

[ Voor 11% gewijzigd door Merethil op 10-06-2023 10:56 ]

[spone]

Het lijkt me trouwens wat onwaarschijnlijk dat je een private key als bearer token gebruikt.

De kans is groter dat je je private key moet gebruiken om een bearer token op te halen, en dat de token die je dan krijgt vervolgens in de authorization header geplaatst moet worden. Maar ik heb geen specifieke kennis van de TransIP API.

En in meer python-algemene zin: je kunt requests' raise_for_status() gebruiken om op statuscodes te checken. Dan hoef je niet iedere mogelijke statuscode af.

[ Voor 26% gewijzigd door spone op 10-06-2023 11:08 ]

[Ben(V)]

@Merethil
Als hij die toe wil voegen aan een bestaande domain ( in dit geval met de naam DOMEIN) heb je gelijk.

En het beste is gewoon response.ok te gebruiken, dan laat je het aan de requests module over.

@spone
Ja hij moet eerst via de auth call een bearer token aanvragen en die kan hij gebruiken en anders calls.

[Merethil]

Ben(V) schreef op zaterdag 10 juni 2023 @ 11:12:
@Merethil
Als hij die toe wil voegen aan een bestaande domain ( in dit geval met de naam DOMEIN) heb je gelijk.

Dat is wel wat 'ie in zijn TS zei en waar de payload die hij als voorbeeld geeft van lijkt te zijn

En het beste is gewoon response.ok te gebruiken, dan laat je het aan de requests module over.

Ah ja, klopt. Ik was even vergeten dat die erin zat; doe niet zoveel met requests in python 😬