Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Add exact match to SSH key name validation regexp #913

Merged
merged 1 commit into from
Nov 10, 2020
Merged

Conversation

vinokurig
Copy link
Contributor

What does this PR do?

Add exact match to SSH key name validation regexp.

Screenshot/screencast of this PR

What issues does this PR fix or reference?

fixes eclipse-che/che#18229

How to test this PR?

  1. Run SSH create key pair for particular host... command.
  2. Enter https://github.com to the quick input.
  3. See: Invalid input alert:
    screenshot-localhost_3030-2020 11 09-15_12_53
  4. Enter github.com to the quick input.
  5. See: key pair successfully created.

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Happy Path Channel

HAPPY_PATH_CHANNEL=stable

@che-bot
Copy link
Contributor

che-bot commented Nov 9, 2020

❌ E2E Happy path tests failed ❗

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia docker.io/maxura/che-theia:913
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:913

Tested with Eclipse Che Single User on K8S (minikube v1.1.1)

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

@che-bot
Copy link
Contributor

che-bot commented Nov 9, 2020

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia docker.io/maxura/che-theia:913
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:913

Tested with Eclipse Che Single User on K8S (minikube v1.1.1)

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

@che-bot
Copy link
Contributor

che-bot commented Nov 10, 2020

❌ E2E Happy path tests failed ❗

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia docker.io/maxura/che-theia:913
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:913

Tested with Eclipse Che Single User on K8S (minikube v1.1.1)

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

@che-bot
Copy link
Contributor

che-bot commented Nov 10, 2020

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia docker.io/maxura/che-theia:913
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:913

Tested with Eclipse Che Single User on K8S (minikube v1.1.1)

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

@vinokurig vinokurig merged commit a4de133 into master Nov 10, 2020
@vinokurig vinokurig deleted the che-18229 branch November 10, 2020 14:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a name validation check when creating a new SSH key pair
3 participants