Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code Climate badges showing 'unknown' and 'not found' #567

Closed
timkurvers opened this issue Nov 19, 2015 · 12 comments
Closed

Code Climate badges showing 'unknown' and 'not found' #567

timkurvers opened this issue Nov 19, 2015 · 12 comments

Comments

@timkurvers
Copy link

Shields.io:

Code Climate
Test Coverage

Code Climate:

Code Climate
Test Coverage

Similar behaviour on http://shields.io/. Apologies if this was reported previously.

@hughrawlinson
Copy link

👍

@espadrine
Copy link
Member

Their main website indicates that the whole site is down for maintenance: https://codeclimate.com. Have you contacted them through Twitter?

@travi
Copy link

travi commented Nov 20, 2015

that may be the case now, but was not when this issue was filed. shields.io badges for code climate have been showing incorrectly (as above) for a few days, but the ones provided directly from code climate were reporting correctly.

@espadrine
Copy link
Member

It appears that we experienced an unexpected spike in db performance that created a bottleneck for a few minutes.

— Nov. 19th, http://status.codeclimate.com

Maybe it is linked?

I'll wait until they resolve their current unscheduled outage to see whether it changes anything.

@espadrine
Copy link
Member

curl -I 'https://codeclimate.com/github/timkurvers/blizzardry.png'

Content-Disposition: inline; filename="/srv/data/codeclimate/app/assets/images/badges/code_climate-4-0.png"

That is the first time I have seen this HTTP header show up like that. It feels wrong.

I contacted them through Twitter here: https://twitter.com/espadrine/status/668209253577039873.

@icambron
Copy link

@espadrine that certainly does seem wrong. In the meantime, seems OK to make the regex a little more accepting:

 var statusMatch = res.headers['content-disposition']
                           .match(/filename="code_climate-(.+)\.png"/);

Could just add .* after the opening quote

@timkurvers
Copy link
Author

Thanks, works like a charm! 😄

@djperrefort
Copy link

I noticed a few days ago that this problem has started again for some of the code climate badges. An example can be found by looking at http://shields.io/.

@paulmelnikow
Copy link
Member

Would you please open a new issue? Thanks!

@paulmelnikow
Copy link
Member

Please also review the open issues: #1329 #1196 #1387.

@rkt2spc
Copy link

rkt2spc commented Mar 20, 2018

Can we reopen this? I'm encountering it now

@RedSparr0w
Copy link
Member

@rocketspacer I'm on mobile right now, but i believe this has been fixed (still needs to be deployed), check issue #1518

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

No branches or pull requests

9 participants