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

Addons can be created without name/summary in the default-locale #8804

Closed
ioanarusiczki opened this issue Jun 16, 2022 · 3 comments · Fixed by mozilla/addons-server#19500
Closed
Assignees
Milestone

Comments

@ioanarusiczki
Copy link

ioanarusiczki commented Jun 16, 2022

STR:

  1. I used a dummy extension with name and summary , also an addon that has default_locale into manifest.json
  2. Create the addon using PUT or POST with the following JSON that contains name and summary in other locale than en-US

{ "categories": { "firefox": [ "other" ] }, "support_email": { "it": "rusiczki.ioana@gmail.com", "ro": "rusiczki.ioana@gmail.com", "en-US": "rusiczki.ioana@gmail.com" }, "developer_comments": { "it": "test" }, "summary": { "it": "en-US" }, "name": { "it": "en-US" }, "description": { "it": "en-US" }, "homepage": { "it": "https://addons-server.readthedocs.io/en/latest/topics/api/addons.html#create", "ro": "https://addons-server.readthedocs.io/en/latest/topics/api/addons.html#create" }, "version": { "upload": "dc4f420f404b4aa5a429d0f6eb442a93", "custom_license": { "name": { "it": "test" }, "text": { "it": "test" } } } }

Expected result:
An error should be thrown - perhaps that the default-locale must be declared in the JSON

Actual result:
The default locale is considered en-US but it has no name or summary in en-US

@ioanarusiczki
Copy link
Author

@eviljeff

I verified on -dev, -stage and this is no longer reproducible. I get a 400 Bad request -> "A value in the default locale of "en-US" is required."

Note: I've noticed with 2 addons a pattern, I tried with Decentraleyes and uBlock Origin from prod an upload and create -> after I get the uuid , the addon create API throws a 400 "Upload not valid" if I try to create immediately. I must wait for a while and then send the request and it will work as expected. I don't know if that's expected. When the addon is a dummy this is not reproducible, but Decentraleyes is 6,7 MB , UBlock 2,8 MB.

@eviljeff
Copy link
Member

@ioanarusiczki

Note: I've noticed with 2 addons a pattern, I tried with Decentraleyes and uBlock Origin from prod an upload and create -> after I get the uuid , the addon create API throws a 400 "Upload not valid" if I try to create immediately. I must wait for a while and then send the request and it will work as expected. I don't know if that's expected. When the addon is a dummy this is not reproducible, but Decentraleyes is 6,7 MB , UBlock 2,8 MB.

#8807 also talked of getting "Upload not valid" - and I couldn't reproduce locally - so it may be the same thing. If it's happening with large, complex, add-ons and not dummy submissions it may be a lag somewhere (database writes?) but it's difficult to say. It sounds like something we'd like to get to understand, at least, so can you log a specific issue for it?

@ioanarusiczki
Copy link
Author

@eviljeff Ok, I'm going to look over #8807 too and reply on it.
Could be the same thing but meanwhile I filed #8832

@KevinMind KevinMind transferred this issue from mozilla/addons-server May 4, 2024
@KevinMind KevinMind added repository:addons-server Issue relating to addons-server migration:2024 labels May 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

3 participants