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

chore: update some dependences, fix some clippy errors #373

Merged
merged 1 commit into from
Feb 1, 2023

Conversation

rjzak
Copy link
Member

@rjzak rjzak commented Feb 1, 2023

Some errors in the output which somehow still allowed the unit test to pass successfully, not sure if intentional or not.

     Running tests/mod.rs (target/debug/deps/mod-839111660b709219)

running 1 test
test app ... 2023-02-01T03:28:29.146903Z ERROR app::auth::oidc: failed to verify token error=Error decoding token

Caused by:
    ExpiredSignature
2023-02-01T03:28:29.151279Z ERROR app::auth::oidc: failed to verify token error=Error decoding token

Caused by:
    InvalidAudience
2023-02-01T03:28:29.159868Z ERROR app::auth::oidc: failed to verify token error=Error decoding token

Caused by:
    InvalidIssuer
2023-02-01T03:28:29.163667Z ERROR app::auth::oidc: failed to verify token error=Error decoding token

Caused by:
    Base64 error: Encoded text cannot have a 6-bit remainder.
ok

The Base64 error occurred after I updated the dependency, but the others were before I touched anything.

@rvolosatovs rvolosatovs marked this pull request as ready for review February 1, 2023 09:07
@rvolosatovs rvolosatovs requested review from a team, bstrie and haraldh as code owners February 1, 2023 09:07
@rvolosatovs rvolosatovs enabled auto-merge (rebase) February 1, 2023 09:07
@rvolosatovs rvolosatovs merged commit 8938e48 into enarx:main Feb 1, 2023
@rvolosatovs
Copy link
Member

Yes, I was confused as well... #323

@rjzak rjzak deleted the deps_clippy_fixes branch July 29, 2023 14:10
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.

2 participants