webcompat.com
webcompat.com copied to clipboard
Use of os.path.normcase causes fetching milestones to fail on Widnows
https://github.com/webcompat/webcompat.com/blob/5714ca924f73fd3ead39cbe29d6f470bcaacd608/config/init.py#L48
Fetching milestones from Github…
It failed with 404 Client Error: Not Found for url: https://api.github.com/repos%5Cwebcompat%5Cwebcompat-tests%5Cmilestones!
We will read from data/milestones.json
Note that https://api.github.com/repos%5Cwebcompat%5Cwebcompat-tests%5Cmilestones is 404 - because %5c is \ and not /. Oops.
I think instead of os.path.normcase, we want to remove that - just os.path.join should work.
In theory we fixed this in https://github.com/webcompat/webcompat.com/issues/2497, but I don't think we actually did. Oops. This is a URI - we don't want any platform-specific handling here.