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

Fixed a bug that locks you out of Quickbase #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

uncleflo
Copy link

@uncleflo uncleflo commented May 20, 2021

When uploading many code page files to Quickbase, sometimes Quickbase would flag suspicious behaviour and block access to your account for 10 minutes.

The reason seems to be...

The package worked by logging the user in implicitly, as part of the upload code page request.

The first few files would upload successfully, however, after a certain number of files are uploaded, Quickbase would flag this as 'suspicious' and lock you out of Quickbase.

Once you get locked out, the rest of the code pages had to be uploaded manually.

This made the library a lot less useful for us.

We also saw some forum posts from other users who had the same issue.

In general this is a really helpful library for us, and we wanted to keep using it. So we decided to fix this issue.

Changes we made...

Previously the package logged the user in implicitly, as part of the upload code page request.

Now, the the user is logged in with an explicit call to the authenticate endpoint.

The ticket returned by Quickbase is cached so it can be reused to upload multiple files.

This avoids authenticating to Quickbase many times, so we no longer get locked out.

In addition:

The library now supports authentication with a user_token as well.

user_token can be specified in config, and also as an env variable.

We've also enabled specifying username and app_token as env variables; previously only password could be an env variable.

@uncleflo uncleflo force-pushed the better_authentication branch from 009eef3 to 2cd5c49 Compare May 20, 2021 19:19
@uncleflo uncleflo force-pushed the better_authentication branch 3 times, most recently from 8e29a03 to 3447377 Compare June 3, 2021 21:21
… activity, plus added support for user_token authentication
@uncleflo uncleflo force-pushed the better_authentication branch from 3447377 to ecc8263 Compare June 8, 2021 18:30
@uncleflo uncleflo changed the title Better Authentication Fixed a bug that locks you out of Quickbase Jun 16, 2021
@uncleflo
Copy link
Author

Hello @cullenjett, we made some enhancements; would you like to have a look and consider reviewing ?

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