search

Vermont-EPSCoR / Macroinvertebrates

Macroinvertebrates Mobile App - Qt Version
MIT License
2 stars 0 forks source link

Use Authenticated API #7

Closed MorganRodgers closed 6 years ago

MorganRodgers commented 8 years ago

Bots are allowed 500 query results per call opposed to the 50 we're currently limited to.

MorganRodgers commented 8 years ago

Human account requested from WikiEducator.

MorganRodgers commented 8 years ago

Note/Reminder that we'll end up having credentials in the application, and that I should NOT commit those credentials to the public repository.

pclemins commented 8 years ago

I suppose we should make it an option to have people enter their own credentials so they can use "bot mode", right?

On Tue, Apr 19, 2016 at 3:56 PM, MorganRodgers notifications@github.com wrote:

Note/Reminder that we'll end up having credentials in the application, and that I should NOT commit those credentials to the public repository.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/MorganRodgers/MacroinvertebratesV3/issues/7#issuecomment-212100536

MorganRodgers commented 8 years ago

We could do that, but I think we'd find that most people wouldn't use it because they wouldn't create accounts with bot access.

Or we could put a place holder file into the repo with a bogus username/password combo and communicate the real combination securely to the person compiling/deploying the application. Here be the dangers of public repos...

pclemins commented 8 years ago

Oh yeah... that would work as well :)

On Tue, Apr 19, 2016 at 8:16 PM, MorganRodgers notifications@github.com wrote:

We could do that, but I think we'd find that most people wouldn't use it because they wouldn't create accounts with bot access.

Or we could put a place holder file into the repo with a bogus username/password combo and communicate the real combination securely to the person compiling/deploying the application. Here be the dangers of public repos...

— You are receiving this because you commented. Reply to this email directly or view it on GitHub https://github.com/MorganRodgers/MacroinvertebratesV3/issues/7#issuecomment-212181500

MorganRodgers commented 8 years ago

I wonder if deploying something like this to the masses would be a violation of ToS? Something to look into.

pclemins commented 8 years ago

I'm not sure this is any worse than what we are doing now... unless you share a common login for the bot across all installs of the field guide. Right now, we actually use more resources as 10 separate 50 bug requests is much more load on the server than a single 500 bug request....

On Tue, Apr 26, 2016 at 4:59 PM, MorganRodgers notifications@github.com wrote:

I wonder if deploying something like this to the masses would be a violation of ToS? Something to look into.

— You are receiving this because you commented. Reply to this email directly or view it on GitHub https://github.com/MorganRodgers/MacroinvertebratesV3/issues/7#issuecomment-214884434