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

Link to zeebe-ssl-certificate flag in the Deploy your first diagram guide #1346

Closed
MaxTru opened this issue Oct 7, 2022 · 14 comments · Fixed by #2052
Closed

Link to zeebe-ssl-certificate flag in the Deploy your first diagram guide #1346

MaxTru opened this issue Oct 7, 2022 · 14 comments · Fixed by #2052
Assignees
Labels
component:modeler Issues related with Modeler project

Comments

@MaxTru
Copy link
Contributor

MaxTru commented Oct 7, 2022

What to do?

Link to zeebe-ssl-certificate flag from the deploy your first diagram guide

Why to do it?

Users in self-managed setups might fail to deploy diagrams from desktop modeler because they have not configured ssl certificate. Probably they would try to find out how to deploy reading this guide.

I think a brief link to the ssl certificate flag in this guide would be helpful to overcome this.

@MaxTru
Copy link
Contributor Author

MaxTru commented Oct 7, 2022

WDYT @barmac ?

@barmac
Copy link
Contributor

barmac commented Oct 10, 2022

I think this makes sense.

@barmac
Copy link
Contributor

barmac commented Oct 10, 2022

I can imagine this as a note on top:

In case you want to deploy to Camunda Platform 8 Self-Managed with a custom SSL certificate, there is a flag which allows to configure it.

@MaxTru
Copy link
Contributor Author

MaxTru commented Oct 10, 2022

Do you know, if this is the default case now for customers that have self-managed with Identity set-up? Or will our self-managed installation with Identity "just work"?

@barmac
Copy link
Contributor

barmac commented Oct 10, 2022

I don't know, I haven't tried Identity setup yet. There is however an issue on deployment problems for this: camunda/camunda-modeler#3152

@MaxTru
Copy link
Contributor Author

MaxTru commented Oct 10, 2022

I think we should try this typical end-2-end use-case first. This will allow us to better organize our Docs.

@rob2universe
Copy link
Contributor

camunda/camunda-modeler#3152 also occurs without ssl

@akeller akeller added the component:modeler Issues related with Modeler project label May 1, 2023
@akeller
Copy link
Member

akeller commented May 1, 2023

@nikku
Copy link
Member

nikku commented May 2, 2023

@akeller Something along these lines? #2052

@akeller
Copy link
Member

akeller commented May 2, 2023

Do you know if we need to mention self-signed certs specifically, or would this apply to any cert? Self-signed certs can make enterprise IT team nervous.

IMO the proposed PR meets the needs and scope of this issue and would close it out.

@nikku
Copy link
Member

nikku commented May 2, 2023

Self-signed certs can make enterprise IT team nervous.

What exactly makes who nervous? Most enterprise IT will have some sort of self-signed root in their chain from which all sorts of trust is inherited. We're seeing this from many customers, across the board: Self-signed root + intermediate + server certificates derived from that.

@nikku
Copy link
Member

nikku commented May 2, 2023

Potential improvement: 5819885.

@akeller
Copy link
Member

akeller commented May 2, 2023

In my experience, getting a cert from a trusted CA is strongly preferred. Self-signed can imply the connection is insecure. Even internally at Camunda, IT asks us not to use self-signed certs. This may be new or due to an update in security scanning tools. This is why I'm asking, just to clarify.

I like your improvement! Gets us away from language that potential customers may focus on as supporting an "insecure" path.

@nikku
Copy link
Member

nikku commented May 2, 2023

Will update the PR (backport changes where necessary). Let's improve this bit and close this issue.

@nikku nikku self-assigned this May 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:modeler Issues related with Modeler project
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants