Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Log URL incorrect due to appended timestamp #8

Open
jezdez opened this issue Jul 9, 2014 · 2 comments
Open

Log URL incorrect due to appended timestamp #8

jezdez opened this issue Jul 9, 2014 · 2 comments

Comments

@jezdez
Copy link
Contributor

jezdez commented Jul 9, 2014

One thing I noticed is that chief seems to append a timestamp to the log URL nowadays, which makes the URL chief-james is showing isn't correct: https://github.com/mozilla/chief/blob/master/chief.py#L40-L45

Added here: mozilla/chief@41af16d

Any idea how to fix this?

@mythmon
Copy link
Owner

mythmon commented Jul 9, 2014

That is problematic. A few ideas, listed in increasing difficulty.

  • Stop linking to logs. 😦
  • Link to the index page.
  • Link to the sorted index page, which is easy to calculate.
  • Parse the html of that sorted index page, and grab the first link, and paste that.

Thoughts?

@jezdez
Copy link
Contributor Author

jezdez commented Jul 10, 2014

@mythmon I think the last bit would be my favorite in the outcome but sadly this would mean adding a HTML parsing depedency like lxml, which sounds like way too much. So in that sense I think the best would be to simply point to the sorted index page.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants