Re: Screenshots.debian.net leaves useless URLs in JSON output

2016-07-19 Thread Andreas Tille
Hi Christoph,

On Tue, Jul 19, 2016 at 09:44:05PM +0200, Christoph Haas wrote:
> 
> The fix is deployed.

Thanks for the quick fix.  I can confirm that the json looks OK.

> You will always get URLs starting with https://… to
> be safe when linking them.

https is fine.
 
> Let me know if you need anything else.

Will do.

Thanks a lot

Andreas.

-- 
http://fam-tille.de



Re: Screenshots.debian.net leaves useless URLs in JSON output

2016-07-19 Thread Andreas Tille
Hi Christoph,

On Tue, Jul 19, 2016 at 12:41:49PM +0200, Christoph Haas wrote:
> sorry for that. Apparently the web application is not aware of its
> real-life URL but rather sees itself behind the caching proxy. I'll fix
> that today.

:-)
 
> The crazy thing is that get requests to
> http://screenshots.debian.net/json/packages every few seconds. Actually
> end-users should be even need that information. Took me a while to
> figure out why the server died from the load until I started caching the
> JSON files. I assume that you are importing the day just once a day, right?

I think UDD is updated once a day (may be twice - to lazy to check).

Kind regards

  Andreas.

-- 
http://fam-tille.de



Re: Screenshots.debian.net leaves useless URLs in JSON output

2016-07-19 Thread Christoph Haas
Hi Andreas…

sorry for that. Apparently the web application is not aware of its
real-life URL but rather sees itself behind the caching proxy. I'll fix
that today.

The crazy thing is that get requests to
http://screenshots.debian.net/json/packages every few seconds. Actually
end-users should be even need that information. Took me a while to
figure out why the server died from the load until I started caching the
JSON files. I assume that you are importing the day just once a day, right?

Cheers
 Christoph


Am 19.07.2016 um 12:05 schrieb Andreas Tille:
> Hi,
>
> for the UDD importer I'm using
>
> http://screenshots.debian.net/json/screenshots
>
> Since some time instead of sensible URLs it leaves things like
>
> http://127.0.0.1:/screenshots/000/008/481/large.png
>
> as large_image_url.  Could you please replace all strings
>
> http://127.0.0.1:/
>
> by
>
> https://screenshots.debian.net/
>
> While I could easily do this inside the UDD importer I think it rather
> should be fixed at the source in case there might be other consumers of
> these data.
>
> Kind regards and thanks for this service
>
>Andreas.
>




signature.asc
Description: OpenPGP digital signature


Screenshots.debian.net leaves useless URLs in JSON output

2016-07-19 Thread Andreas Tille
Hi,

for the UDD importer I'm using

http://screenshots.debian.net/json/screenshots

Since some time instead of sensible URLs it leaves things like

http://127.0.0.1:/screenshots/000/008/481/large.png

as large_image_url.  Could you please replace all strings

http://127.0.0.1:/

by

https://screenshots.debian.net/

While I could easily do this inside the UDD importer I think it rather
should be fixed at the source in case there might be other consumers of
these data.

Kind regards and thanks for this service

   Andreas.

-- 
http://fam-tille.de