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

Modified table class name for non-cached UserAgent #110

Closed
wants to merge 1 commit into from

Conversation

Jordan9675
Copy link

It seems like the class attribute associated to the table on https://www.w3schools.com/browsers/default.asp has changed thus leading to IndexError when trying to initialize UserAgent with cache=False

@thebetauser
Copy link

Confirmed this fixes it the issues, need one of the admins to merge this request to main branch

@toh995
Copy link

toh995 commented Jul 26, 2021

Thank you for catching this, hopefully it gets merged soon 🙏

@pmartynov
Copy link

@hellysmile who is managing this repo? I need this fix to get merged.

@peterschutt
Copy link

@RapidEU fork it and do it yourself.. no one is obligated to do anything for you.

Copy link

@maxkrivich maxkrivich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Jordan9675
Copy link
Author

Jordan9675 commented Sep 21, 2021

https://github.com/hellysmile/fake-useragent/blob/460686d0c729c2b22172dc75760408c87b7d6618/fake_useragent/settings.py#L7

Do not forget to update the version.

Thanks for pointing this out, I changed it and added it within the commit

@jacebrowning jacebrowning mentioned this pull request Nov 1, 2021
@MattDMo
Copy link

MattDMo commented Nov 6, 2021

bump....

@Treeless
Copy link

Please merge this Admins!!!

@arunkannawadi
Copy link

The error you see is not an actual error but a traceback from an exception that is handled internally. You should be able to ignore the apparent error and continue running the code. However, if you'd like to suppress the annoying/confusing message , you can temporarily redirect your output into the void using this nice little trick:
https://thesmithfam.org/blog/2012/10/25/temporarily-suppress-console-output-in-python/

@lucawen
Copy link

lucawen commented Dec 17, 2021

this repo is dead

@MattDMo
Copy link

MattDMo commented Dec 27, 2021

@arunkannawadi No, this is an actual error that prevents the module from generating a list of browser User-Agent strings. Hiding the error isn't going to make it go away, you need to update the code as detailed in this PR.

michaelhball added a commit to michaelhball/fake-useragent that referenced this pull request Feb 12, 2022
@melroy89
Copy link
Collaborator

melroy89 commented Mar 31, 2022

Maybe we should fork it and create a new pip package. I see that @hellysmile is from Ukraine.

I will do it: https://github.com/danger89/fake-useragent

@MattDMo
Copy link

MattDMo commented Jul 30, 2022

@hellysmile 's last contribution to Github was on 2022-04-14, and there has been no activity since. During the previous year, he was fairly consistent in his activity, with the longest lag being about a month in Nov-Dec 2021. How long do we wait before petitioning PyPI to change the fake_useragent project to use @Danger89 's fork? Is everyone okay with that fork? I am.

I just sent an email to hellysmile@gmail.com, asking Victor if he intends to come back to this repo, or if he'd mind if @Danger89 take over in some manner. I suggest waiting a few weeks or a month for a reply, and if I don't hear from him we can put together the PEP 541 info.

@melroy89
Copy link
Collaborator

melroy89 commented Aug 1, 2022

I'm fine with that as well.

@sgerodes
Copy link

I am ok with that too.

@MattDMo
Copy link

MattDMo commented Oct 14, 2022

Well, it's been over 2 months, and I haven't heard a thing, so it looks like @hellysmile is either unwilling or unable to respond, for whatever reason. @Danger89, would you like to start the petitioning process with PyPI? Feel free to let me know if you need help. I don't have a ton of time to devote to this, but I'd like to see it through to completion.

@melroy89
Copy link
Collaborator

Well, it's been over 2 months, and I haven't heard a thing, so it looks like @hellysmile is either unwilling or unable to respond, for whatever reason. @Danger89, would you like to start the petitioning process with PyPI? Feel free to let me know if you need help. I don't have a ton of time to devote to this, but I'd like to see it through to completion.

Thanks for your time and trying to reach out to @hellysmile.

I will look into the PyPi transfer procedure and create an issue report soon at there side. Let's do this!

@melroy89
Copy link
Collaborator

FYI. Transfer ticket created: pypi/support#2313

@melroy89
Copy link
Collaborator

melroy89 commented Nov 2, 2022

@hellysmile Looks like he is back. I'm now owner of the repo as well as the pypi package.

I will merge my changes back upstream #136

@melroy89 melroy89 closed this Nov 2, 2022
@chrisspen
Copy link

Still getting this error in 1.4.0. Any chance of getting a new release?

@melroy89
Copy link
Collaborator

Are you sure you updated the right package and are on V1. 4.0? Since all this table code and cache is gone. We don't scrape data like that anymore. We also have all user agent data locally stored within the package.

pip install --upgrade fake-useragent

@chrisspen
Copy link

Ah, you're right. I had upgraded, but I was activating the wrong virtual environment. Thanks.

@melroy89
Copy link
Collaborator

Np.

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