remove trever for user list

FRC5892 / HeroHours

Team 5892's attendance solution

1 stars 4 forks source link

remove trever for user list #36

Open MichaelStatt-S opened 1 month ago

MichaelStatt-S commented 1 month ago

fixes #35

[x] add endpoint for member sheet
[x] implement basic auth for sheet endpoint
[ ] add permissions to use sheet
[x] make custom function to use basic auth in Google sheets
[ ] add endpoint for user log
[x] add endpoint for by-meeting using url parameters

MichaelStatt-S commented 1 month ago

[ ] add activity logs (mostly copy and paste)

yyy2356 commented 1 month ago

id like to deploy these changes on the test deployment to test everything and make sure it all works before we merge. will do the review when we do that

MichaelStatt-S commented 1 month ago

It almost does, however, I need to implement it myself to use basic authentication. I could put it in the url, but that is even less secure. The headers for basic authentication are at least encrypted with https, the url isn't. It should have the same functionality besides the headers, it acts the exact same

yyy2356 commented 1 month ago

why dont we set it up the way openai does, where the request has a json header and the credentials can be part of the content. alternatively we could create api keys. for example we could use djangos REST for either method, or we could implement a custom middleware

MichaelStatt-S commented 1 week ago

This should now be good. Now a priority because trever is broken