Noitidart / Chrome-Store-Foxified

Point and click to install extensions into your browser from the AMO, Chrome Web Store, Microsoft Store, or Opera Addons websites.
https://addons.mozilla.org/en-US/firefox/addon/chrome-store-foxified/?src=github
Mozilla Public License 2.0
365 stars 56 forks source link

Randomize signing URL names #73

Open Madis0 opened 7 years ago

Madis0 commented 7 years ago

The problem:

The solution:

What do you think?

Noitidart commented 7 years ago

This is an important one. I forgot to land this. I'll do this before v3 gets out, I was about to release. I'll put in the modal and then this first.

Noitidart commented 7 years ago

I'm exploring here - https://addons-server.readthedocs.io/en/latest/topics/api/addons.html#detail - i haven't found a way to change the slug yet, still looking. Only way so far is to change addon name to something ugly, but then that will show up with the ugly and non-descriptive name in the addon manager, I think. Would be awesome if you can help me peak into this API.

Madis0 commented 7 years ago

Isn't that it?

addon-id – The id for the add-on. https://addons-server.readthedocs.io/en/latest/topics/api/signing.html#uploading-a-version

Noitidart commented 7 years ago

It seems that id is not used for the slug. :(

As seen in this screenshot, I prefixed the UUID of "File upload" kind with "generated" but that only becomes the uuid, the slug is taken from the name it seems:

Madis0 commented 7 years ago

What about randomizing the name on upload but changing it right after? I don't think that name will show up on add-on manager anyway, that should use the one set in manifest.

Noitidart commented 7 years ago

That's realy smart! So do 2x review processes instead of one, on the first one use a random name, then on the second one change it. That's a possibility, its more then a small change to the current flow. :( It might be easier to go to the edit page and figure post to the form (non-rest api style) (web scraping style).

The name in the review is also taken from manifest.

Croydon commented 7 years ago

AFAIK the AMO team wants to change this anyway. Only public listed add-ons should get a slug in the future. No idea how fast this will get implemented.

Noitidart commented 7 years ago

That is a super helpful comment, if they do that, then I don't have to do anything woohoo haha.