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

The last cookie in the config is not used by the script #5

Open
na-ji opened this issue Dec 28, 2019 · 3 comments
Open

The last cookie in the config is not used by the script #5

na-ji opened this issue Dec 28, 2019 · 3 comments

Comments

@na-ji
Copy link

na-ji commented Dec 28, 2019

Hello,

I had trouble running the script, but I finally found why after some debugging.

I followed the readme and copied my cookie like it was asked. But I kept running in the error Oops!, looks like you have a problem with your cookie..
After some debugging, the error is because the API does not use the last cookie given. It does not use it because the regex is looking for a semi-colon, and the last cookie does not have a semi-colon.

In my case it was a problem because the last cookie is the most important one:
image

I fixed my problem by adding a final semi-colon in the config, but I think other users might run into this problem too.

Thanks.

@SenorKarlos
Copy link

my cookie has no ; at the end and works, when ingress isn't being catankerous. I had it shut off for a bit due to my use of bash and webhook spam, but testing now shows it back on and fetching. Maybe you had bad timing? Your example image also shows it ending in a " after sessionid whereas mine continues with a semicolon form that point and ends at zoom level.

@na-ji
Copy link
Author

na-ji commented Dec 28, 2019

In my case, the last cookie in the list is the most important one, the session id. In your case it's the the zoom level (a useless cookie). So it's not bad timing, but bad luck. It all depends on the order of the cookies sent by Ingress.
If you want to understand why the regex does not select the last cookie:
image
https://regex101.com/r/fSIp9N/2

@SenorKarlos
Copy link

Ah ok. Purely observational comment, I don't know what's important or not in that copypaste, just that it said to take it all and mine works. I'd assume that it's either ignoring the parts it doesn't need or the fact my 'zoom cookie' has no ; doesn't bother the program. The regex link does help highlight what you're saying tho :) thanks

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

No branches or pull requests

2 participants