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

[FR - CNEWS] Fix Error 403 #1447

Closed
wants to merge 1 commit into from

Conversation

jeff2900
Copy link
Contributor

No description provided.

@joaopa00
Copy link
Contributor

Thanks for the patch. But a better way to fix this problem is to update the user-anegst globally. I pushed a patch for that.

@joaopa00 joaopa00 closed this Dec 17, 2024
@jeff2900
Copy link
Contributor Author

Ok as you want. I keep patch for my use.

@jeff2900 jeff2900 deleted the dev-cnews branch December 17, 2024 15:03
@joaopa00
Copy link
Contributor

J'ai réutilisé ton patch. Certaines distribs ne sont pas à jour au niveaux des certificats. Et CNEWS est très pointilleux sur ce sujet (ainsi que la version des navigateurs)

Evidemment, je t'ai laissé comme auteur du patch.

@jeff2900
Copy link
Contributor Author

Merci @joaopa00,
En attendant une hypothétique mise à jour de la bibliothèque urlquick, une solution de contournement serait de remplacer urlquick par requests comme dans le fichier mycnal.py

J'ai utilisé un en-tête fixe à la place de l'en-tête aléatoire car l'en-tête aléatoire provoquait aléatoirement l'erreur 403.
Je ne sais pas quel en-tête entre Chrome, Edg, Safari et Vivaldi est à l'origine de l'erreur 403. Ne voulant pas modifier le fichier web_utils.py, j'ai opté pour l'en-tête fixe.

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.

2 participants