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

[NO MERGE] Registration Code #109

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

[NO MERGE] Registration Code #109

wants to merge 8 commits into from

Conversation

A1Liu
Copy link
Collaborator

@A1Liu A1Liu commented Sep 18, 2022

DO NOT MERGE THIS

NYU's rules on account health/safety are not explicit on whether or not third parties may register for courses on your behalf, but pretty strongly imply that admin does not condone sharing your username and password with third parties (e.g. Schedge), and also seem to imply that admin wouldn't be very excited with its students maintaining an application which has access to other students' logins.

Of course, you can use this code as inspiration for future projects that execute locally from your own computer, but this code cannot be added to Schedge proper, because even if NYU doesn't prohibit it explicitly, we should not poke the bear.

See https://www.nyu.edu/about/policies-guidelines-compliance/policies-and-guidelines/netid-accounts-access.html for more information.

  • Albert Liu (original creator of Schedge)

P.S. As an additional aside, even if you NYU allowed us to register on behalf of students, this code uses a bypass around MFA which the original author found. Don't merge this.

Credits

This code was written by @Kn99HN , but wasn't used in Schedge due to concerns about NYU's policies on account safety. It was later removed in order to prevent any kind of future compliance problems.

@A1Liu A1Liu marked this pull request as draft September 18, 2022 03:16
@A1Liu A1Liu changed the title Registration Code [NO MERGE] Registration Code Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant