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

Coverity badge always gets "inaccessible" status #3334

Closed
1 of 3 tasks
mity opened this issue Apr 18, 2019 · 12 comments · Fixed by #3336
Closed
1 of 3 tasks

Coverity badge always gets "inaccessible" status #3334

mity opened this issue Apr 18, 2019 · 12 comments · Fixed by #3336
Labels
question Support questions, usage questions, unconfirmed bugs, discussions, ideas service-badge New or updated service badge

Comments

@mity
Copy link

mity commented Apr 18, 2019

Are you experiencing an issue with...

🪲 Description

The Coverity Scan badge for MD4C project always shows in a gray color with the status message as "inaccessible". Noticed this few days ago for the first time. But before that it used to work correctly for a long time.

I've just checked https://scan.coverity.com/projects/mity-md4c/badge.json and it seems to get the following JSON correctly:

{"project":"mity/md4c","status":"passed","message":"passed"}

🔗 https://img.shields.io/coverity/scan/mity-md4c.svg?label=coverity%20scan

@calebcartwright
Copy link
Member

Thanks @mity taking a look now, will update when we have more info

@calebcartwright
Copy link
Member

calebcartwright commented Apr 18, 2019

Looks like there's an issue in the cert chain, below is the error we're getting internally:

Inaccessible: unable to verify the first certificate

Very similar underlying issue to #1956

https://www.ssllabs.com/ssltest/analyze.html?d=scan.coverity.com&hideResults=on

@calebcartwright calebcartwright added question Support questions, usage questions, unconfirmed bugs, discussions, ideas service-badge New or updated service badge labels Apr 18, 2019
@mity
Copy link
Author

mity commented Apr 18, 2019

I see.

Well, the question is whether it makes any sense to be stricter then browsers are.

@calebcartwright
Copy link
Member

Well, the question is whether it makes any sense to be stricter then browsers are.

This isn't due to anything we're doing explicitly. It's just the default behavior of many http clients. Why the browsers operate the way they do vs. those http clients is an entirely separate conversation 😄

The underlying issue is related to what sounds like a recently introduced issue in Coverity's certificate chain. We're mulling some options on if/how we could potentially work around such issues in the upstream services (like Coverity). However, ideally those services would resolve their certificate issues vs. us having to disable/relax ssl validation on our end.

@mity
Copy link
Author

mity commented Apr 18, 2019

This isn't due to anything we're doing explicitly. It's just the default behavior of many http clients. Why the browsers operate the way they do vs. those http clients is an entirely separate conversation 😄

Ok.

The underlying issue is related to what sounds like a recently introduced issue in Coverity's certificate chain. We're mulling some options on if/how we could potentially work around such issues in the upstream services (like Coverity). However, ideally those services would resolve their certificate issues vs. us having to disable/relax ssl validation on our end.

I can fully understand that. But my attempts to contact them (for some other minor issue like the service not working at all for some project) all failed, and I never got any response from them, so I am skeptical. :-(

@calebcartwright
Copy link
Member

calebcartwright commented Apr 18, 2019

Understood. I'll make sure that if we add a work around on our end, that we include that support for our Coverity badge as well.

@jarun
Copy link

jarun commented Apr 19, 2019

This affects one of my projects (bcal) too.

@calebcartwright
Copy link
Member

All Coverity badges are going to show as inaccessible due to an issue in the Coverity service's cert chain.

We may deploy a work around in the near future, but I'd still suggest that folks try to get in touch with Coverity to see if they can resolve those cert issues

@jarun
Copy link

jarun commented Apr 19, 2019

Thanks for the quick response. I'll drop a mail right away. However, their response time is painfully slow in my experience.

@calebcartwright
Copy link
Member

calebcartwright commented Apr 20, 2019

FYI we've updated things on our end so that requests for Coverity badges will explicitly disable the strict ssl check, thus restoring Coverity badges.

Once #3336 is deployed to prod, your badges will automatically start working again (this comment will track the status of that deployment).

For a visual, here's the two Coverity badges referenced above from our staging app:

https://shields-staging.herokuapp.com/coverity/scan/mity-md4c.svg?label=coverity%20scan

https://shields-staging.herokuapp.com/coverity/scan/jarun-bcal.svg

@jarun
Copy link

jarun commented Apr 20, 2019

Thank you!

1 similar comment
@jarun
Copy link

jarun commented Apr 20, 2019

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Support questions, usage questions, unconfirmed bugs, discussions, ideas service-badge New or updated service badge
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants