Travis CI Flaw Exposed Secrets of Thousands of Open Source Projects (arstechnica.com) 28
An anonymous reader quotes a report from Ars Technica: Travis CI is a popular software-testing tool due to its seamless integration with GitHub and Bitbucket. As the makers of the tool explain: "When you run a build, Travis CI clones your GitHub repository into a brand-new virtual environment and carries out a series of tasks to build and test your code. If one or more of those tasks fail, the build is considered broken. If none of the tasks fail, the build is considered passed and Travis CI can deploy your code to a web server or application host." But this month, researcher Felix Lange found a security vulnerability that caused Travis CI to include secure environment variables of all public open source repositories that use Travis CI into pull request builds. Environment variables can include sensitive secrets like signing keys, access credentials, and API tokens. If these variables are exposed, attackers can abuse the secrets to obtain lateral movement into the networks of thousands of organizations.
Tracked as CVE-2021-41077, the bug is present in Travis CI's activation process and impacts certain builds created between September 3 and September 10. As a part of this activation process, developers are supposed to add a ".travis.yml" file to their open source project repository. This file tells Travis CI what to do and may contain encrypted secrets. Another place encrypted secrets may be defined is Travis' web UI. But, these secrets are not meant to be exposed. In fact, Travis CI's docs have always stated, "Encrypted environment variables are not available to pull requests from forks due to the security risk of exposing such information to unknown code." Ideally, Travis is expected to run in a manner that prevents public access to any secret environment variables specified. [...] This vulnerability caused these sorts of secrets to be unexpectedly exposed to just about anyone forking a public repository and printing files during a build process. Fortunately, the issue didn't last too long -- around eight days, thanks to Lange and other researchers who notified the company of the bug on September 7. But out of caution, all projects relying on Travis CI are advised to rotate their secrets.
The presence and relatively quick patching of the flaw aside, Travis CI's concise security bulletin and overall handling of the coordinated disclosure process has infuriated the developer community. In a long Twitter thread, Peter Szilagyi details the arduous process that his group endured as it waited for Travis CI to take action and release a brief security bulletin on an obscure webpage. "After 3 days of pressure from multiple projects, [Travis CI] silently patched the issue on the 10th. No analysis, no security report, no post mortem, not warning any of their users that their secrets might have been stolen," tweeted Szilagyi. After Szilagyi and Lange asked GitHub to ban Travis CI over its poor security posture and vulnerability disclosure processes, an advisory showed up. "Finally, after multiple ultimatums from multiple projects, [they] posted this lame-ass post hidden deep where nobody will read it... Not even a single 'thank you.' [No] acknowledgment of responsible disclosure. Not even admitting the gravity of it all," said Szilagyi, while referring to the security bulletin -- and especially its abridged version, which included barely any details. Szilagyi was joined by several members of the community in criticizing the bulletin. Boston-based web developer Jake Jarvis called the disclosure an "insanely embarrassing 'security bulletin.'" "Travis CI implemented a series of security patches starting on Sept 3rd that resolves this issue," concluded Mendy on behalf of the Travis CI team. "As a reminder, cycling your secrets is something that all users should do on a regular basis. If you are unsure how to do this, please contact Support."
Tracked as CVE-2021-41077, the bug is present in Travis CI's activation process and impacts certain builds created between September 3 and September 10. As a part of this activation process, developers are supposed to add a ".travis.yml" file to their open source project repository. This file tells Travis CI what to do and may contain encrypted secrets. Another place encrypted secrets may be defined is Travis' web UI. But, these secrets are not meant to be exposed. In fact, Travis CI's docs have always stated, "Encrypted environment variables are not available to pull requests from forks due to the security risk of exposing such information to unknown code." Ideally, Travis is expected to run in a manner that prevents public access to any secret environment variables specified. [...] This vulnerability caused these sorts of secrets to be unexpectedly exposed to just about anyone forking a public repository and printing files during a build process. Fortunately, the issue didn't last too long -- around eight days, thanks to Lange and other researchers who notified the company of the bug on September 7. But out of caution, all projects relying on Travis CI are advised to rotate their secrets.
The presence and relatively quick patching of the flaw aside, Travis CI's concise security bulletin and overall handling of the coordinated disclosure process has infuriated the developer community. In a long Twitter thread, Peter Szilagyi details the arduous process that his group endured as it waited for Travis CI to take action and release a brief security bulletin on an obscure webpage. "After 3 days of pressure from multiple projects, [Travis CI] silently patched the issue on the 10th. No analysis, no security report, no post mortem, not warning any of their users that their secrets might have been stolen," tweeted Szilagyi. After Szilagyi and Lange asked GitHub to ban Travis CI over its poor security posture and vulnerability disclosure processes, an advisory showed up. "Finally, after multiple ultimatums from multiple projects, [they] posted this lame-ass post hidden deep where nobody will read it... Not even a single 'thank you.' [No] acknowledgment of responsible disclosure. Not even admitting the gravity of it all," said Szilagyi, while referring to the security bulletin -- and especially its abridged version, which included barely any details. Szilagyi was joined by several members of the community in criticizing the bulletin. Boston-based web developer Jake Jarvis called the disclosure an "insanely embarrassing 'security bulletin.'" "Travis CI implemented a series of security patches starting on Sept 3rd that resolves this issue," concluded Mendy on behalf of the Travis CI team. "As a reminder, cycling your secrets is something that all users should do on a regular basis. If you are unsure how to do this, please contact Support."
Open Source Secrets (Score:4, Insightful)
That almost sounds like an oxymoron.
Re: (Score:1)
Well, it sure is on GitHub, anyway. I am Jack's complete lack of surprise.
Re: Open Source Secrets (Score:2)
Travis not GitHub (Score:2)
This is Travis, not GitHub. They released a blog entry [travis-ci.com] which doesn't even link to their own bulletin. Starting to move off Travis at this point would make sense. This is not trustworthy behaviour. Moving from GitHub to GitLab would also make sense but that's not related to this problem - instead it's because GitLab has an Open Source edition and so you can self host and/or fix later if you wish.
Re: (Score:3)
"Information wants to be free."
Re: (Score:1)
That almost sounds like an oxymoron.
Only if two conditions are met:
1. One does not understand open source.
2. One is a moron.
an oxy - moron?
Re: (Score:1)
It's also consistent with Richard M. Stallman's attitudes about security. He was infamous at MIT for setting his password to blank, or when compelled to have a password to set his account "rms" to the password "rms", and to leave computer science systems he might have access to for open access because of just such attitudes. It's what got him kicked out of the computer labs at Kendall Square, part of MIT's facilities where he'd been a brilliant, but very eccentric, guest for years.
rms is.... not a moron, bu
Re: (Score:2)
Re: (Score:1)
refusing to admit that rules have some uses and leaving devastation in his wake.
"rules" that aren't applied to everyone equally are injustice. Most people supporting BLM or even supporting "defund the police" are not against rules, they're against a corrupt institution that has ignore the rules for far too long. All we ask if that if you drag a woman out of her car to beat her senselessly, that you don't use her child as a prop on social media [yahoo.com]
Example of Cognative Dissonance (Score:3)
Two things (Score:5, Insightful)
1) Don't check secrets into source control, even if they are encrypted. The encryption you used might already be broken, and you don't follow the right crypto forums to be notified when it does get broken.
2) Don't make your CI tools accessible on the public internet. Put it behind a VPN because they all have security bugs.
Re: (Score:2)
"So don't use encryption, use an encrypted network." That's your advice?!
Re: (Score:2)
There are two reasons:
1) "only use one encryption system" instead of one for your secrets, one for your internet traffic, etc. It's simpler, there's less room for human error.
2) The VPN prevents unauthenticated traffic from even touching your CI server, and breaking through that is a more difficult attack than merely reading an encrypted message.
Not a fan of CI/CD (Score:2)
Works as designed (Score:2)
If this had happened because a programmer had made a mistake, it would be that programmer's responsibility. But, because they're using Travis CI, the blame is deflected to someone else.
The fact of the matter is that security holes are almost impossible to avoid on any project of non-trivial complexity. The reason why third-party systems like this are used is because it allows blame to be shifted to someone else in the event of a security breach. And it's working - yes, the vendor looks bad, but a secur
Too much diversity :] (Score:1)
Wow, talk about brazen BS (Score:2)
"Travis CI implemented a series of security patches starting on Sept 3rd that resolves this issue"
They're actually trying to credit the introduction of the bug as part of the solution to the bug, to screw up the dates and make it look like a same-day response.
Lame and sleazy.
I'm glad I went with a more professional CI system for my own code. Travis CI was the very first one I evaluated, because I prefer Ruby for high-level code. But it just didn't have the features I wanted.
Testing Tool (Score:1)
Re: (Score:1)
Re: (Score:1)
Re: (Score:1)
Re: (Score:1)
Re: (Score:1)
Re: (Score:1)
Re: (Score:1)
Re: (Score:1)