Skip to content

Fix credential state crash#541

Merged
Dindexx merged 3 commits intoopenwallet-foundation-labs:developfrom
lissi-id:bugfix/fix-credential-state-crash
Apr 30, 2026
Merged

Fix credential state crash#541
Dindexx merged 3 commits intoopenwallet-foundation-labs:developfrom
lissi-id:bugfix/fix-credential-state-crash

Conversation

@Dindexx
Copy link
Copy Markdown
Contributor

@Dindexx Dindexx commented Apr 27, 2026

Summary

Program crashed during refresh credential state due to unhandled exception. This PR fixes that.

Related Issue(s)

If applicable link the related issue(s).

Checklist

  • No secrets, credentials, or sensitive data are added
  • CHANGELOG.md updated under [Unreleased] with correct category and links according to CONTRIBUTING.md
  • README.md updated if applicable
  • (Unit) Tests added if applicable

@Dindexx Dindexx requested a review from JoTiTu April 27, 2026 19:01
@Dindexx Dindexx self-assigned this Apr 27, 2026
<PackageReference Include="jose-jwt" Version="5.0.0"/>
<PackageReference Include="LanguageExt.Core" Version="4.4.9"/>
<PackageReference Include="Microsoft.Extensions.Http" Version="3.1.5"/>
<PackageReference Include="Microsoft.Extensions.Logging" Version="$(MicrosoftExtensionsLoggingVersion)"/>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why using the build.props?

@JoTiTu JoTiTu self-requested a review April 30, 2026 09:40
Copy link
Copy Markdown
Contributor

@JoTiTu JoTiTu left a comment

Choose a reason for hiding this comment

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

Fix merge request then LGTM.

Dindexx added 3 commits April 30, 2026 20:06
Signed-off-by: Kevin <kevin.dinh@lissi.id>
Signed-off-by: Kevin <kevin.dinh@lissi.id>
Signed-off-by: Kevin <kevin.dinh@lissi.id>
@Dindexx Dindexx force-pushed the bugfix/fix-credential-state-crash branch from 3272ff1 to 032a2bc Compare April 30, 2026 18:07
@Dindexx Dindexx merged commit a2f0b4a into openwallet-foundation-labs:develop Apr 30, 2026
3 checks passed
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