Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0056305Openbravo ERPA. Platformpublic2024-08-26 12:232024-08-29 11:11
Reporternjimenez 
Assigned Tonjimenez 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionPR24Q4 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0056305: Improvements on oauth2 for api implementation
DescriptionSome improvements are going to be added on oauth2 implementation for openbravo api.

- Improve OAuth2TokenAuthenticationManagerTest test implementation
- Add test to StandaloneTestSuite
- Check if the Configuration class inside OAuthTokenConfigProvider can be setted as protected
Steps To Reproduce.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0056007main closed ignacio_deandres OAuth2 Authentication for Openbravo API 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2024-08-26 12:23njimenezNew Issue
2024-08-26 12:23njimenezAssigned To => njimenez
2024-08-26 12:23njimenezOBNetwork customer => No
2024-08-26 12:23njimenezModules => Core
2024-08-26 12:23njimenezTriggers an Emergency Pack => No
2024-08-28 13:01njimenezDescription Updatedbug_revision_view_page.php?rev_id=28384#r28384
2024-08-28 13:18hgbotMerge Request Status => open
2024-08-28 13:18hgbotNote Added: 0168514
2024-08-29 11:08hgbotMerge Request Statusopen => approved
2024-08-29 11:10hgbotNote Added: 0168543
2024-08-29 11:10hgbotResolutionopen => fixed
2024-08-29 11:10hgbotStatusnew => closed
2024-08-29 11:10hgbotFixed in Version => PR24Q4
2024-08-29 11:10hgbotNote Added: 0168544
2024-08-29 11:11caristuRelationship addedrelated to 0056007

Notes
(0168514)
hgbot   
2024-08-28 13:18   
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/1339 [^]
(0168543)
hgbot   
2024-08-29 11:10   
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/1339 [^]
(0168544)
hgbot   
2024-08-29 11:10   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/openbravo [^]
Changeset: 61835a69ce5db82b833f7a88ea1a927e488acb8b
Author: Natalia Jimenez <natalia.jimenez@openbravo.com>
Date: 29-08-2024 09:10:32
URL: https://gitlab.com/openbravo/product/openbravo/-/commit/61835a69ce5db82b833f7a88ea1a927e488acb8b [^]

Fixed ISSUE-56305: Improve OAuth2 token authentication test

  - Test improvements
  - It was missing to include the test into the test suite
  - Reduce the visibility of the Configuration class to the minimum necessary

---
M src-test/src/org/openbravo/authentication/oauth2/OAuth2TokenAuthenticationManagerTest.java
M src-test/src/org/openbravo/test/StandaloneTestSuite.java
M src/org/openbravo/authentication/oauth2/OAuthTokenConfigProvider.java
---