Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Issue 29. Add basic authentication quarantine #35

Merged
merged 2 commits into from
Aug 14, 2024
Merged

Conversation

ksenia-vazhdaeva
Copy link
Contributor

No description provided.

@ksenia-vazhdaeva
Copy link
Contributor Author

Based on #34

@ksenia-vazhdaeva ksenia-vazhdaeva changed the title WIP: Issue 29. Add basic authentication quarantine Issue 29. Add basic authentication quarantine Aug 9, 2024
@ksenia-vazhdaeva ksenia-vazhdaeva requested a review from bwsw August 9, 2024 08:51

.. code-block:: bash

echo "{
Copy link
Contributor

Choose a reason for hiding this comment

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

Please replace with here docs everything with massive backslash jsons/yamls.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What is meant by here docs?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in #34

chmod 644 certs/ca.key certs/server.key certs/client.key

``certs/*.crt`` is a file with a certificate in PEM format and ``certs/*.key`` is a file with a key in PEM format where ``*`` is a replacement for `ca`, `server` and `client`.

Launching etcd
Copy link
Contributor

Choose a reason for hiding this comment

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

Change to Etcd (capital letter). Check other places too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I used the same writing as in the official documentation - etcd. See https://etcd.io/

@@ -241,10 +321,6 @@ The configuration below does not contain TLS settings for simplicity. For produc
\"urls\": [
Copy link
Contributor

Choose a reason for hiding this comment

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

here-doc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in #34

@ksenia-vazhdaeva ksenia-vazhdaeva merged commit f17cbfe into develop Aug 14, 2024
1 check passed
@ksenia-vazhdaeva ksenia-vazhdaeva deleted the issue-29 branch August 14, 2024 09:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add quarantine for a client not passing the authentication/authorization
2 participants