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

RFC3161HashRepository accepts rfc3161_provider only as a string and Pydantic URLs are not strings anymore #3281

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

Donnype
Copy link
Contributor

@Donnype Donnype commented Jul 23, 2024

Changes

Bugfix that is rather trivial.

Issue link

Closes #3277

Demo

QA notes

To enable an rfc provider add these test values to your env, restart kat and make sure you can add an OOI (also see the issue description):

RFC3161_PROVIDER=https://freetsa.org/tsr
RFC3161_CERT_FILE=bytes/timestamping/certificates/freetsa.crt

Code Checklist

  • All the commits in this PR are properly PGP-signed and verified.
  • This PR only contains functionality relevant to the issue.
  • I have written unit tests for the changes or fixes I made.
  • I have checked the documentation and made changes where necessary.
  • I have performed a self-review of my code and refactored it to the best of my abilities.

Checklist for code reviewers:

Copy-paste the checklist from the docs/source/templates folder into your comment.


Checklist for QA:

Copy-paste the checklist from the docs/source/templates folder into your comment.

@Donnype Donnype requested a review from a team as a code owner July 23, 2024 10:43
@underdarknl underdarknl added the backport Needs to be backported to previous release label Jul 23, 2024
@underdarknl underdarknl merged commit f3a3f92 into main Jul 23, 2024
9 checks passed
@underdarknl underdarknl deleted the fix/bytes-signing-provider-url-pydantic branch July 23, 2024 11:22
dekkers pushed a commit that referenced this pull request Jul 23, 2024
jpbruinsslot added a commit that referenced this pull request Jul 25, 2024
* fix/mula/new_boefjes:
  Add more tests
  Add additional thread safety test for the other caches
  Add thread safety tests
  Fix new boefjes issue
  feat: Updated color scheme (#3241)
  Bump sphinx from 7.4.6 to 7.4.7 (#3265)
  Fix rocky logging (#3288)
  Add XTDB list and rename method in origin tool (#3234)
  RFC3161HashRepository accepts rfc3161_provider only as a string and Pydantic URLs are not strings anymore (#3281)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Needs to be backported to previous release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bytes exception when timestamping is enabled
2 participants