fix: Http client exception not handled properly resulting in incorrectly formatted error

[2shrestha22]

Pull Request

• Report security issues confidentially.
• Any contribution is under this license.
• Link this pull request to an issue.

Issue

Closes: #1022

Approach

Tasks

• Add tests
• Add changes to documentation (guides, repository pages, code comments)

[parse-github-assistant]

Thanks for opening this pull request!

[mbfakourii]

We use two clients, dio and http, at Parse Flutter !

In this change you are using the functions of the http package. Will this not cause any problems when we use the dio package client?

[2shrestha22]

DioException is already being handled above but ClientException was missing.

[mbfakourii]

@mtrezza

Can you please run CI?

[mtrezza]

@mbfakourii done

[codecov]

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 43.47%. Comparing base (c388545) to head (65f624e).
Report is 8 commits behind head on master.

Files with missing lines	Patch %	Lines
...src/objects/response/parse_exception_response.dart	33.33%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1021      +/-   ##
==========================================
+ Coverage   43.37%   43.47%   +0.10%     
==========================================
  Files          61       61              
  Lines        3463     3466       +3     
==========================================
+ Hits         1502     1507       +5     
+ Misses       1961     1959       -2     

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

[parse-github-assistant]

I will reformat the title to use the proper commit message syntax.

[mtrezza]

I rephrased the PR title; not sure whether it accurately describes the issue, could you please review?

[2shrestha22]

I think it's better.

[mtrezza]

There is still an open review discussion and the changelog entry + version bump is missing in the PR. See other PRs on how they are done. Then we can go ahead and merge.