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

Allowing to configure if you wish the process to exit when blacklisted #52

Merged
merged 1 commit into from
Jun 6, 2018

Conversation

Scotsoo
Copy link
Contributor

@Scotsoo Scotsoo commented May 29, 2018

I'm not too keen on the process exiting when my API key is invalid ect. This should be configurable as far as I'm concerned.

@cnguy cnguy changed the base branch from master to dev May 31, 2018 01:00
@cnguy
Copy link
Owner

cnguy commented May 31, 2018

Hi, thanks for the PR. Changes look good, but instead of exitOnRateLimitExceed, which is a 429, a different name should be used. shouldExitOn403? shouldExitOnForbiddenKey? What do you think?

@cnguy cnguy merged commit c95b6c3 into cnguy:dev Jun 6, 2018
@Scotsoo
Copy link
Contributor Author

Scotsoo commented Jun 7, 2018

Will do update this now. Not had chance to get some code done over the last week.

@Scotsoo
Copy link
Contributor Author

Scotsoo commented Jun 7, 2018

Scotsoo@b93a7ef

@Scotsoo
Copy link
Contributor Author

Scotsoo commented Jun 7, 2018

Oh, I see you updated this: 4cfc9e1

@cnguy
Copy link
Owner

cnguy commented Jun 7, 2018

Sorry about that, forgot to update you! I merged your changes though since they were basically complete and just renamed the stuffs. I'll publish later.

@cnguy
Copy link
Owner

cnguy commented Jun 7, 2018

I'll make sure to update the console.log that you added too so thanks for that

@cnguy
Copy link
Owner

cnguy commented Jun 8, 2018

Published to 0.8.15! Thank you once again for the work + suggestion. @Scotsoo

@Scotsoo
Copy link
Contributor Author

Scotsoo commented Jun 8, 2018

No worries. Yeah, I realised the console.log wouldn't log cause the process would've exited.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants