Skip to content

Fix self signed certificate for iPad #2255

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

Merged
merged 6 commits into from
Nov 3, 2020
Merged

Fix self signed certificate for iPad #2255

merged 6 commits into from
Nov 3, 2020

Conversation

nhooyr
Copy link
Contributor

@nhooyr nhooyr commented Oct 30, 2020

See composing commits.

@nhooyr nhooyr requested a review from code-asher October 30, 2020 08:14
@nhooyr nhooyr force-pushed the self-signed-3b2c branch 2 times, most recently from 7f6f2bb to bd2d1b5 Compare October 30, 2020 08:41
@nhooyr
Copy link
Contributor Author

nhooyr commented Oct 30, 2020

Second commit here is very juicy!

@nhooyr nhooyr changed the title util: Generate self signed certificate into data directory Fix self signed certificate for iPad Oct 30, 2020
@nhooyr nhooyr force-pushed the self-signed-3b2c branch 2 times, most recently from 4b41f62 to c8daba3 Compare October 30, 2020 17:00
Now we add a subject alt name, set extendedKeyUsage and use the
correct certificate extension.

The above allow it to be properly trusted by iOS.

See https://support.apple.com/en-us/HT210176

*.cert isn't a real extension for certificates, *.crt is correct
for it to be recognized by e.g. keychain or when importing as a profile
into iOS.

Updates #1566

I've been able to successfully connect from my iPad Pro now to my
code-server instance with a self signed certificate! Next commit
will be docs.
Copy link
Member

@code-asher code-asher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is 🔥, wonderful commit separation too.

doc/ipad.md Outdated

The below assumes you are using the self signed certificate that code-server
generates for you. If not, that's fine but you'll have to make sure your certificate
abides by the following guidelines from apple: https://support.apple.com/en-us/HT210176
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
abides by the following guidelines from apple: https://support.apple.com/en-us/HT210176
abides by the following guidelines from Apple: https://support.apple.com/en-us/HT210176

@nhooyr nhooyr merged commit f9e0990 into master Nov 3, 2020
@nhooyr nhooyr deleted the self-signed-3b2c branch November 3, 2020 19:04
@nhooyr nhooyr mentioned this pull request Nov 16, 2020
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.

2 participants