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

chore: force https in react storybook deployment #5078

Merged

Conversation

vpicone
Copy link
Contributor

@vpicone vpicone commented Jan 17, 2020

The carbondesignsystem.com ssl cert now uses a wildcard. We should force the react storybook to leverage https and avoid the "Connection not secure" notices from browsers.

@vpicone vpicone requested a review from a team as a code owner January 17, 2020 00:29
@netlify
Copy link

netlify bot commented Jan 17, 2020

Deploy preview for the-carbon-components ready!

Built with commit 0f6c797

https://deploy-preview-5078--the-carbon-components.netlify.com

@netlify
Copy link

netlify bot commented Jan 17, 2020

Deploy preview for carbon-elements ready!

Built with commit 0f6c797

https://deploy-preview-5078--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Jan 17, 2020

Deploy preview for carbon-components-react failed.

Built with commit 0f6c797

https://app.netlify.com/sites/carbon-components-react/deploys/5e22546d710552000b50b6a2

@joshblack
Copy link
Contributor

joshblack commented Jan 17, 2020

@vpicone did anything change in how we upload the cert to IBM Cloud? We've uploaded wildcard certs in the past and they didn't really work 😞 Ignore me! Tested https://react.carbondesignsystem.com/?path=/story/accordion--default and it worked perfectly.

@vpicone
Copy link
Contributor Author

vpicone commented Jan 17, 2020

@joshblack you could also deploy to a new cf app to make sure the option is working as intended.

@vpicone vpicone merged commit 8810405 into carbon-design-system:master Jan 20, 2020
@vpicone vpicone deleted the force-https-react-storybook branch January 20, 2020 16:07
joshblack added a commit to joshblack/carbon that referenced this pull request Jan 23, 2020
…m#5078)

Co-authored-by: Josh Black <josh@josh.black>
Co-authored-by: Akira Sudoh <asudoh@gmail.com>
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

Successfully merging this pull request may close these issues.

3 participants