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

Flickr User Agent requirement !!! #695

Closed
Marfir opened this issue Mar 22, 2023 · 5 comments
Closed

Flickr User Agent requirement !!! #695

Marfir opened this issue Mar 22, 2023 · 5 comments
Assignees

Comments

@Marfir
Copy link

Marfir commented Mar 22, 2023

Subject of the issue

"We will begin to require a user agent header on all API calls. Specifically, your calls will need to provide a “User-Agent” header, and the header needs to have a value that roughly describes your client. A good header for a client might look like “User-Agent: MySetManager/2.3” or “User-Agent: PHP_Flickr/1.0”. We believe this change will help us better manage traffic and improve our overall API availability."

see https://www.flickr.com/groups/api/discuss/72157721918374433/

Your environment

  • version of Flickr4Java 3.0.6
  • version of JDK 11

Expected behaviour

User-Agent header is set in class "REST" inside of the "get" Method. The name and the version of the App needs to be injected (e.g. by method parameter, class constructor parameter etc.).

e.g. request.addHeader("User-Agent", AppNameFromSomewhere + "/" + AppVersionFromSomewhere);

Actual behaviour

no User-Agent is set

@Marfir
Copy link
Author

Marfir commented Mar 22, 2023

@boncey @callmeal
Please can you adopt the required API change? Flickr team will be test it in March and April before they try to block any requests without a User-Agent header.

@boncey
Copy link
Owner

boncey commented Mar 22, 2023

Hello @Marfir. Thanks for letting me know about this.
Unfortunately, I’m on holiday at the moment without a laptop so this will have to wait until I get back in a week or so.

I will look at it when I return.

@boncey boncey self-assigned this Mar 22, 2023
@Marfir
Copy link
Author

Marfir commented Mar 23, 2023

Thx for your super fast response!
No problem. Have a nice holiday :-)

@Marfir
Copy link
Author

Marfir commented Mar 24, 2023

Update
After a shit storm from the community they step back from their plan:

`Dear valued community members,

We wanted to reach out to you to apologize for the recent announcement of a new requirement for requiring User-agents to API requests. We understand that this news may have caused concern and inconvenience, and we wanted to take a moment to address your feedback.

After careful consideration and review of your feedback, we have decided not to proceed with the launch of this requirement. We recognize that it would have had a significant impact on existing implementations and legacy systems. Our team had hoped that this requirement would have had a positive impact on our community, but we understand that it would have been too disruptive for many of our users.

We take responsibility for not fully understanding the impact of this requirement before announcing it, and we sincerely apologize for any confusion or disruption it may have caused. We are committed to listening to your feedback and doing our best to meet the needs of our community.

Once again, we apologize for any inconvenience caused, and we thank you for your understanding and continued support.`

But I prefer to add the User-Agent header independent of their recent announcement. It's a small and easy change and this lib will be save for the future.

@boncey
Copy link
Owner

boncey commented Apr 30, 2023

You can now provide a User-Agent or else it uses the default. Available in version 3.0.7 Details in the README.

@boncey boncey closed this as completed Apr 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants