Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0049159
TypeCategorySeverityReproducibilityDate SubmittedLast Update
feature request[Openbravo ERP] A. Platformminorhave not tried2022-04-28 09:542022-04-29 14:18
ReportercaristuView Statuspublic 
Assigned Tocaristu 
PriorityhighResolutionfixedFixed in VersionPR22Q3
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0049159: Add a new authentication method that authenticates by just setting the Authorization header with basic authentication

DescriptionCurrently we have the BasicHttpAuthorizationProvider which follows the standard HTTP basic authentication[1].

It would be nice to have a different authorization provider to authenticate with an external system that does not strictly follow the standard HTTP authentication but expects to directly have the basic authentication in the Authorization header of the request.

[1] https://developer.mozilla.org/en-US/docs/Web/HTTP/Authentication [^]
Steps To Reproduce.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to feature request 0048579 closedcaristu Push API 
causes defect 0049171 closedcaristu External system with Basic Auth (Always in Header) does not allow to enter user and password 

-  Notes
(0136835)
hgbot (developer)
2022-04-28 11:29

Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/578 [^]
(0136841)
hgbot (developer)
2022-04-28 18:38

Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/578 [^]
(0136842)
hgbot (developer)
2022-04-28 18:38

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/openbravo [^]
Changeset: a4bcd39d50f179bbad42fb3aeba950b50fdf8d1e
Author: Carlos Aristu <carlos.aristu@openbravo.com>
Date: 28-04-2022 13:17:32
URL: https://gitlab.com/openbravo/product/openbravo/-/commit/a4bcd39d50f179bbad42fb3aeba950b50fdf8d1e [^]

fixes ISSUE-49159: new "set always the basic auth in header" auth provider

  Provide a new authorization provider to authenticate against external
systems which do not follow the standard HTTP authentication but expect
the basic auth information to be always present in the request header.

  Note that for the encoding of the basic credentials we are using the
same solution the standard JDK implementation of the Authenticator
applies when authenticating with basic credentials.

---
A src/org/openbravo/service/externalsystem/http/BasicHttpAlwaysInHeaderAuthorizationProvider.java
A src/org/openbravo/service/externalsystem/http/HttpAuthorizationRequestHeaderProvider.java
M src-db/database/sourcedata/AD_REF_LIST.xml
M src-test/src/org/openbravo/service/externalsystem/http/HttpExternalSystemCommunicationTest.java
M src/org/openbravo/service/externalsystem/http/BasicHttpAuthorizationProvider.java
M src/org/openbravo/service/externalsystem/http/HttpExternalSystem.java
---

- Issue History
Date Modified Username Field Change
2022-04-28 09:54 caristu New Issue
2022-04-28 09:54 caristu Assigned To => Triage Platform Base
2022-04-28 09:54 caristu Modules => Core
2022-04-28 09:54 caristu Triggers an Emergency Pack => No
2022-04-28 09:54 caristu Assigned To Triage Platform Base => caristu
2022-04-28 09:54 caristu Status new => acknowledged
2022-04-28 09:54 caristu Status acknowledged => scheduled
2022-04-28 09:55 caristu Relationship added related to 0048579
2022-04-28 10:11 caristu Summary Support setting HTTP request headers in authorization methods => Add a new authentication method that authenticates setting the Authorization header with basic authentication
2022-04-28 10:11 caristu Description Updated View Revisions
2022-04-28 10:12 caristu Summary Add a new authentication method that authenticates setting the Authorization header with basic authentication => Add a new authentication method that authenticates just setting the Authorization header with basic authentication
2022-04-28 10:12 caristu Description Updated View Revisions
2022-04-28 11:29 hgbot Note Added: 0136835
2022-04-28 11:56 caristu Summary Add a new authentication method that authenticates just setting the Authorization header with basic authentication => Add a new authentication method that authenticates by just setting the Authorization header with basic authentication
2022-04-28 18:38 hgbot Resolution open => fixed
2022-04-28 18:38 hgbot Status scheduled => closed
2022-04-28 18:38 hgbot Note Added: 0136841
2022-04-28 18:38 hgbot Fixed in Version => PR22Q3
2022-04-28 18:38 hgbot Note Added: 0136842
2022-04-29 08:33 alostale Relationship added causes 0049171
2022-04-29 14:18 ivancaceres Issue Monitored: ivancaceres


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker