Skip to content

feat: update dependencies and fix API compatibility issues#122

Merged
hsluoyz merged 4 commits intomasterfrom
copilot/update-dependencies-and-fix-issues
Feb 16, 2026
Merged

feat: update dependencies and fix API compatibility issues#122
hsluoyz merged 4 commits intomasterfrom
copilot/update-dependencies-and-fix-issues

Conversation

Copy link
Contributor

Copilot AI commented Feb 16, 2026

fix: #121

The SDK fails to deserialize responses from recent Casdoor versions due to missing fields and lacks dependency declarations for annotations in use.

Changes

API Compatibility

  • Added @JsonIgnoreProperties(ignoreUnknown = true) to all entity classes and CasdoorResponse to handle API evolution
  • Added sub field to CasdoorResponse for new API response format

Dependency Updates

  • Jackson: 2.13.3 → 2.18.2
  • OkHttp: 4.9.3 → 4.12.0
  • Nimbus JOSE JWT: 9.10 → 9.41.1
  • JUnit Jupiter: 5.8.2 → 5.11.4
  • Added JSR-305 (3.0.2) for nullability annotations previously undeclared

Annotations

Replaced JetBrains annotations (not in pom.xml) with JSR-305:

  • org.jetbrains.annotations.NotNulljavax.annotation.Nonnull
  • org.jetbrains.annotations.Nullablejavax.annotation.Nullable

Breaking Change with Backward Compatibility

Created correctly-spelled Provider class. Provier is now deprecated and extends Provider to maintain compatibility:

@Deprecated
public class Provier extends Provider {
    // Existing code continues to work
}

Users should migrate to Provider in future updates.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@hsluoyz hsluoyz marked this pull request as ready for review February 16, 2026 02:29
@codecov
Copy link

codecov bot commented Feb 16, 2026

Codecov Report

❌ Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (942509d) to head (0dff9d3).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
.../main/java/org/casbin/casdoor/entity/Provider.java 0.00% 10 Missing ⚠️
.../org/casbin/casdoor/util/http/CasdoorResponse.java 0.00% 3 Missing ⚠️
...c/main/java/org/casbin/casdoor/entity/Provier.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master    #122   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files          93      94    +1     
  Lines        1390    1397    +7     
  Branches       36      36           
======================================
- Misses       1390    1397    +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI and others added 3 commits February 16, 2026 02:33
…add sub field to CasdoorResponse

Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
…tation (jsr305)

Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
…rd compatibility

Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
Copilot AI changed the title [WIP] Update outdated dependencies and fix related issues Update dependencies and fix API compatibility issues Feb 16, 2026
Copilot AI requested a review from hsluoyz February 16, 2026 02:40
@hsluoyz hsluoyz changed the title Update dependencies and fix API compatibility issues feat: update dependencies and fix API compatibility issues Feb 16, 2026
@hsluoyz hsluoyz merged commit a29532f into master Feb 16, 2026
4 of 5 checks passed
@github-actions
Copy link

🎉 This PR is included in version 1.41.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Outdated dependencies (and other issues)

3 participants

Comments