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.

Sign up for GitHub

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

Jump to bottom

clean up and remove auth from code #4

Closed
wants to merge 1 commit into from
Closed

clean up and remove auth from code #4

wants to merge 1 commit into from

Conversation

gmchad
Copy link

@gmchad gmchad commented Jan 25, 2023

  1. Gets basic auth out of plaintext and into an env file
  2. Refactor to a class structure

@gmchad gmchad mentioned this pull request Jan 25, 2023
Open
@gmchad
Copy link
Author

gmchad commented Jan 26, 2023

@TheAppleTucker @evanmays let's get this merged?

@evanmays
Copy link
Collaborator

This net adds LOC w/o adding new features and removes the ability for anyone to use our keys :(

We'd like people to use the keys until the free credits run out

@evanmays evanmays closed this Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@japko1980 japko1980 japko1980 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gmchad @evanmays @japko1980