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

Bugfix/ai avatar #6591

Open
wants to merge 4 commits into
base: DevelNUI
Choose a base branch
from
Open

Conversation

AnglerLee
Copy link
Collaborator

Description of Change

  • Implemented IDisposable for RestClient and properly disposed of HttpClient to resolve dispose defect (WID:9875287).
  • Improved exception handling by analyzing the root cause of NullReferenceException and applying defensive programming to prevent it from occurring (WID:9861897).

API Changes

  • ACR: None

Angler added 4 commits January 20, 2025 13:13
- RestClient now implements IDisposable and disposes the HttpClient instance.
- ServiceClientManager now properly disposes all RestClient instances.
- Added Dispose pattern to RestClient to ensure HttpClient is disposed correctly.
- Updated ServiceClientManager to use using statement or call Dispose method to release resources.

WID:9875283
- Analyzed the root cause of NullReferenceException and modified the code to check for null objects and perform appropriate actions.
- Improved exception handling by catching more specific exception types instead of NullReferenceException, making the exception handling clearer and allowing separate handling for specific exception types.
- Applied defensive programming to validate input parameters and check for null objects to prevent NullReferenceException from occurring.

WID:9861897
@TizenAPI-Bot
Copy link
Collaborator

Internal API Changed

Added: 1, Removed: 0, Changed: 0

Added

+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.AIAvatar.RestClient::Dispose(System.Boolean)

Copy link
Contributor

@hinohie hinohie left a comment

Choose a reason for hiding this comment

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

Checked. Minor. Please add prefix [AIAvatar] for the name of PR.

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

Successfully merging this pull request may close these issues.

3 participants