search

csrdelft / csrdelft.nl

De webstek van Civitas Studiosorum Reformatorum, wordt onderhouden door de PubCie.
https://csrdelft.nl
18 stars 11 forks source link

Voeg een endpoint toe voor de PrakCie teller API #1172

Closed NathanHuisman closed 5 months ago

NathanHuisman commented 5 months ago

Authentication wil ik misschien nog uitvogelen, maar het is geen groot probleem als deze API publiek is. Misschien zorgen dat het IP dat van confide matcht oid?

sonarcloud[bot] commented 5 months ago

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

1 New issue
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

qurben commented 5 months ago

@NathanHuisman Als makkelijke oplossing kan je zoiets doen als in

https://github.com/csrdelft/csrdelft.nl/blob/6f96aeb25a102a663ca3055851324d1c33be58df/htdocs/bar/controller/Barsysteem.class.php#L27-L29

Niet het allernetst, maar die secret is sowieso al geladen op de confide pc.

Voor de nette manier zou je oauth kunnen doen, maar dat is dan weer gezeur aan de client kant. Maar op zich zou je lange tokens moeten kunnen genereren die je kan verversen.