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

#519 Query Certificates with subjectKeyId #532

Merged

Conversation

Abdulbois
Copy link
Collaborator

@Abdulbois Abdulbois commented Jan 10, 2024

  • Enable querying certs with subjectKeyId
  • Add migration flow (pki module - from 2 to 3 version)
  • Update OpenApi yaml file (includes missed changes from PID scoped Accounts #523)

@Abdulbois Abdulbois force-pushed the #519-Query-PAA-certificate-with-only-subjectKeyId branch 30 times, most recently from 74c1028 to 3a4dca7 Compare January 11, 2024 11:35
@@ -0,0 +1,1660 @@
#!/bin/bash
# Copyright 2020 DSR Corporation
Copy link
Contributor

Choose a reason for hiding this comment

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

Discussed with Aziz:
It would be better to have a more realistic upgrade test which perform update from 0.12 to 1.2 and then 1.2 to 1.3.
As for test readability, we can still have multiple modules (files) for each version, and the master test will call all the modules one by one.
It's OK to fix it in a separate PR.

@Abdulbois Abdulbois force-pushed the #519-Query-PAA-certificate-with-only-subjectKeyId branch from 361168a to 554aa94 Compare January 18, 2024 11:46
@Abdulbois Abdulbois force-pushed the #519-Query-PAA-certificate-with-only-subjectKeyId branch 13 times, most recently from fbca6af to d03de3b Compare January 18, 2024 17:00
docs/transactions.md Outdated Show resolved Hide resolved
x/pki/handler_test.go Show resolved Hide resolved
@Abdulbois Abdulbois force-pushed the #519-Query-PAA-certificate-with-only-subjectKeyId branch 5 times, most recently from f649fa0 to c2c9e78 Compare January 19, 2024 06:58
Enable querying certs with subjectKeyId
Add migration flow (from 2 to 3 version of `pki` module)

Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois123@gmail.com>
@Abdulbois Abdulbois force-pushed the #519-Query-PAA-certificate-with-only-subjectKeyId branch from c2c9e78 to fa1bd32 Compare January 19, 2024 07:31
@akarabashov akarabashov merged commit 4f1f627 into master Jan 22, 2024
8 checks passed
@akarabashov akarabashov deleted the #519-Query-PAA-certificate-with-only-subjectKeyId branch March 7, 2024 14:39
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