631735 - Getting a 403 sometimes on image upload, when trying to poll for preview thumbnails of screenshots

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Developer Pages, defect, P3)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Still seeing the 403 on preview, when trying to upload screenshots (during the polling for the thumbnails):

[17:14:12.241] GET https://addons-cdn.allizom.org/img/uploads/previews/thumbs/53/53035.png?modified=1296868429 [HTTP/1.1 403 Forbidden 7ms]
[17:14:04.439] POST https://addons.allizom.org/en-US/developers/addon/a9-9331/upload_preview [HTTP/1.1 200 OK 2644ms]
[17:14:07.395] POST https://addons.allizom.org/en-US/developers/addon/a9-9331/upload_preview [HTTP/1.1 200 OK 356ms]
[17:14:08.703] POST https://addons.allizom.org/en-US/developers/addon/a9-9331/edit_media/edit [HTTP/1.1 200 OK 869ms]
[17:14:09.566] GET https://addons-cdn.allizom.org/img/uploads/previews/thumbs/53/53035.png?modified=1296868429 [HTTP/1.1 403 Forbidden 114ms]
[17:14:09.607] GET https://addons-cdn.allizom.org/img/uploads/previews/thumbs/53/53036.png?modified=1296868429 [HTTP/1.1 200 OK 114ms]
[17:14:09.636] GET https://addons.allizom.org/en-US/developers/ajax/addon/a9-9331/image/status?_=1296868449632 [HTTP/1.1 200 OK 170ms]
[17:14:09.780] GET https://addons-cdn.allizom.org/img/uploads/previews/thumbs/53/53036.png?modified=1296868429&1296868449739 [HTTP/1.1 200 OK 10ms]
[17:14:09.868] GET https://addons-cdn.allizom.org/media/img/addon-icons/default-64.png?1296868449837 [HTTP/1.1 200 OK 11ms]
[17:14:09.894] GET https://addons-cdn.allizom.org/media/img/addon-icons/default-32.png?1296868449842 [HTTP/1.1 200 OK 4ms]

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Screenshot

Comment 2

[Jeff Balogh (:jbalogh)]

(In reply to comment #0)
> Still seeing the 403 on preview, when trying to upload screenshots (during the
> polling for the thumbnails):
> 
> [17:14:12.241] GET
> https://addons-cdn.allizom.org/img/uploads/previews/thumbs/53/53035.png?modified=1296868429
> [HTTP/1.1 403 Forbidden 7ms]
> ....
> [17:14:09.566] GET
> https://addons-cdn.allizom.org/img/uploads/previews/thumbs/53/53035.png?modified=1296868429
> [HTTP/1.1 403 Forbidden 114ms]
> [17:14:09.607] GET
> https://addons-cdn.allizom.org/img/uploads/previews/thumbs/53/53036.png?modified=1296868429
> [HTTP/1.1 200 OK 114ms]

The cache-buster could use a bit more entropy.

Comment 3

[Wil Clouser [:clouserw]]

(In reply to comment #2)

> The cache-buster could use a bit more entropy.

would that fix this bug?

You guys were talking about this on IRC, so I'm not sure if there is more context here.  I'm giving this to Andy since he's been doing the polling stuff, but say something here if this bug is already solved.

Comment 4

[Andy McKay]

Don't think there's anything to worry about the 403, that's an error to Image, which doesn't distinguish between 404 and 403.

I've added more entropy to the checking, just in case.

https://github.com/jbalogh/zamboni/commit/44faa7be0b1fa62b9d8d6a3e68440db157ebd6d9

Comment 5

[Stephen Donner [:stephend] Not actively reading bugmail]

Still getting a 403, but this hasn't regressed anything either, that I can tell.