Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Login should use POST and submit the username and password as part of the body #21

Open
ExtremoPenguin opened this issue Jul 5, 2022 · 3 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@ExtremoPenguin
Copy link

Login should use POST and submit the username and password as part of the request body rather than in the query string. Using GET and query string params for logins will be blocked by the Infinite Campus application in the near future.

@qwazwsx
Copy link
Collaborator

qwazwsx commented Jul 5, 2022

Thanks for your contribution & welcome to GitHub!

We can definitely change that, it's a super simple change. However, I'm curious how you know that GET logins are being deprecated by Infinite Campus. Are you connected with I.C. in any way?

Also: someone else brought up a similar issue earlier, see #17

@ExtremoPenguin
Copy link
Author

You were correct in the other issue that HTTPS hides the query string. So the data is hidden in flight, but full urls often show up in other places such as server logs. Using POST bodies to send sensitive data is generally preferred to reduce the number of places were sensitive data may show up.

@qwazwsx
Copy link
Collaborator

qwazwsx commented Jul 6, 2022

Hm, that's an angle that I hadn't considered. In #17 I really only looked at it from the point of view of a MITM attack, but I hadn't considered server logs. As @arachno7 stated in the above-mentioned issue, switching to POST is basically a drop-in replacement. I'll try to push this change with #20 later this week.

I'd love to keep maintaining this library but unfortunately, I no longer have access to an Infinite Campus account that is enrolled in any courses. This has made trying to test PRs and solve issues very difficult. I'll try my best to get this done in a reasonable amount of time. The way I see it, the only way this project can stay alive is either if
a) other contributors take over (although this just kicks the can further down the line)
or
b) I somehow get access to an Infinite Campus account that's actively enrolled in courses. I'd love it if Infinite Campus gave me some kind of test/demo account that I can use for the purposes of developing this library. But I doubt that'd ever happen.

Anyways, thanks for the heads up, I really appreciate people contributing to this little project. :)

@qwazwsx qwazwsx added enhancement New feature or request help wanted Extra attention is needed labels Jul 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants