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

32p: create second .yml for cookies login with 2fa #8491

Closed
wants to merge 2 commits into from

Conversation

janot
Copy link
Contributor

@janot janot commented May 2, 2020

I'm still being kicked out after some hours with 2fa even with lockip disabled. No such problem with cookies login.
I browsed through repo and looks like creating additional .yml is the way in such cases? Until this is implemented #7780

@jackettbot
Copy link

jackettbot bot commented May 2, 2020

Hi @janot,

Thanks for your contribution to Jackett!

If you are adding a new indexer, please ensure that you've added it to the readme as well

A human will be along soon to review

@janot
Copy link
Contributor Author

janot commented May 2, 2020

Not a new indexer

@garfield69
Copy link
Contributor

before I merge this, take a second look at the regular 32pages indexer.
the login page on the site has the following

<input type="checkbox" id="keeplogged" name="keeplogged" value="1"/>
<label for="keeplogged">Remember me</label>

but the indexer sets the following

@janot
Copy link
Contributor Author

janot commented May 2, 2020

@garfield69 omg, sorry. I'll check, how it behaves with this one for several days

@janot
Copy link
Contributor Author

janot commented May 5, 2020

Works as expected now

@janot janot closed this May 5, 2020
@janot janot deleted the 32p branch May 5, 2020 13:19
janot added a commit to janot/Jackett that referenced this pull request May 5, 2020
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.

None yet

2 participants