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

Resolved401Error #31

Closed
wants to merge 1 commit into from
Closed

Resolved401Error #31

wants to merge 1 commit into from

Conversation

vidishraj
Copy link

Resolved 401 code error.
Related to issue: index_pe_pb_div results in JSONDecodeError #30 Added cookies to header that are fetched from the main domain

Resolved 401 code error. 
Related to issue: index_pe_pb_div results in JSONDecodeError aeron7#30
Added cookies to header that are fetched from the main domain
@vivekguptta
Copy link

I tried your fork and used nsefetch(url).
It gives JSONDecodeError one time and it works properly the next time.
It seems to working properly for every alternate request. First time it gets 401 and then 200 response.

@vidishraj
Copy link
Author

@vivekguptta
Hey bro, I refactored the code and found a workaround for my main project. Never updated the fork.
I think the maintainer has already pushed a similar solution based on refreshed cookies
Please try it out, otherwise let me know, I’ll push my code as well!

@aeron7
Copy link
Owner

aeron7 commented Nov 28, 2023

I had a fucked a previously working code on someone's comment and as I use server edition. I did not check it further. Just unfucked it. https://forum.unofficed.com/t/nse-results-giving-jsondecodeerror/1315/7

@aeron7 aeron7 closed this Nov 28, 2023
@vidishraj vidishraj deleted the patch-3 branch March 31, 2024 04:53
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.

3 participants