Skip to content

Fix NPE in error response parsing#448

Merged
cdancy merged 1 commit intocdancy:masterfrom
hpoettker:fix-npe-in-errors
Feb 12, 2025
Merged

Fix NPE in error response parsing#448
cdancy merged 1 commit intocdancy:masterfrom
hpoettker:fix-npe-in-errors

Conversation

@hpoettker
Copy link
Contributor

Resolves #447

I've successfully tested the commit based on the branch of #446. The mock tests are successful and it works in my application against a real-world Bitbucket instance.

This PR here is against the master branch, which currently doesn't build. I'll rebase after #446 has been merged.

@cdancy cdancy self-requested a review February 10, 2025 14:50
@cdancy cdancy added the bug label Feb 10, 2025
@cdancy cdancy merged commit da0e293 into cdancy:master Feb 12, 2025
@hpoettker hpoettker deleted the fix-npe-in-errors branch February 12, 2025 14:48
@cdancy
Copy link
Owner

cdancy commented Feb 12, 2025

@hpoettker if you're here poking around and working on bitbucket, and have some time, feel free to check out some of the other PR's that have been long standing and are open and see if they still work and make sense to merge. They could be a win for the community. I currently don't have the time I used to in order to dedicate to this project and so any help here is appreciated.

@hpoettker
Copy link
Contributor Author

@cdancy I'd be happy to contribute more fixes and additions. But this really only makes sense when there are also releases to Maven Central. Otherwise, maintaining a private fork is really simpler.

I don't feel strongly about #452 or any other open issue or PR. But a release of the current master would be very much appreciated.

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.

NullPointerException when parsing error responses

2 participants