-
Notifications
You must be signed in to change notification settings - Fork 1
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
CI: Run and report tests on push #17
Conversation
0f56d7e
to
3e99595
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
||
val Base64UrlStrict = Base64(config = Base64ConfigBuilder().apply { | ||
lineBreakInterval = 0 | ||
encodeToUrlSafe = true | ||
isLenient = true | ||
padEncoded = false | ||
}.build()) | ||
|
||
|
||
val Base64Strict = Base64(config = Base64ConfigBuilder().apply { | ||
lineBreakInterval = 0 | ||
encodeToUrlSafe = false | ||
isLenient = true | ||
padEncoded = true | ||
}.build()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is it not used everywhere?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this covered in #18 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, that was very misleading and sloppy! Why don't we want such a shorthand also for Base16?
But come to think of it: Kotlin 1.9 ships with base64 and hex encoding. We might not need the library any more
This comment was marked as outdated.
This comment was marked as outdated.
I think this PR is ready to merge, please review |
8b1139e
to
d1ebc94
Compare
This implementas a rudimentary DER encoder for X.509 certificates
d1ebc94
to
d2d4fd4
Compare
No description provided.