Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0035171 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
feature request | [Openbravo ERP] A. Platform | minor | have not tried | 2017-02-06 17:24 | 2017-03-15 20:19 | |||
Reporter | mtaal | View Status | public | |||||
Assigned To | mtaal | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | 3.0PR17Q2 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 200a7c792da2 | ||||
Projection | none | ETA | none | Target Version | 3.0PR17Q2 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | alostale | |||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0035171: Support basic authentication in the default authentication manager | |||||||
Description | The OB authentication manager supports basic authentication in case of webservice calls [1]. For other authentication we support login/password in the request parameters [2] but we don't support basic authentication. [1] https://code.openbravo.com/erp/devel/pi/file/01c864252395/src/org/openbravo/authentication/AuthenticationManager.java#l479 [^] [2] https://code.openbravo.com/erp/devel/pi/file/01c864252395/src/org/openbravo/authentication/basic/DefaultAuthenticationManager.java#l87 [^] | |||||||
Steps To Reproduce | Do a retail webservice request using basic authentication. | |||||||
Proposed Solution | Support basic authentication in the DefaultAuthenticationManager [1] [1] https://code.openbravo.com/erp/devel/pi/file/01c864252395/src/org/openbravo/authentication/basic/DefaultAuthenticationManager.java#l87 [^] | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|||||||||
|
![]() |
|
(0094140) hgbot (developer) 2017-02-08 09:38 |
Repository: tools/automation/pi-mobile Changeset: c55fdc10cb2447a5d75887ba82739f7be9ceb27a Author: Martin Taal <martin.taal <at> openbravo.com> Date: Wed Feb 08 09:38:12 2017 +0100 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/c55fdc10cb2447a5d75887ba82739f7be9ceb27a [^] Related to issue 35172 and issue 35171: added testcases for stateless/basic authentication Use new class TestExternalOrderLoader to validate on the server that requests are indeed stateless. Explicitly add basic authentication headers --- M src-test/org/openbravo/test/mobile/retail/extmodules/unittest/suites/StatelessTestSuite.java M src-test/org/openbravo/test/mobile/retail/mobilecore/webservice/WebServicesHelper.java M src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/BaseRetailOrderLoaderTest.java A src-test/org/openbravo/test/mobile/retail/extmodules/unittest/tests/statelesswebservice/StatelessRetailOrderLoaderTest.java A src-test/org/openbravo/test/mobile/retail/extmodules/unittest/tests/statelesswebservice/message-order-minimal-1.json A src-test/org/openbravo/test/mobile/retail/extmodules/unittest/tests/statelesswebservice/message-order-multiline-1.json --- |
(0094141) hgbot (developer) 2017-02-08 09:42 |
Repository: erp/devel/pi Changeset: 200a7c792da248caf9ae395d117904c3e5f1fb83 Author: Martin Taal <martin.taal <at> openbravo.com> Date: Wed Feb 08 09:40:46 2017 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/200a7c792da248caf9ae395d117904c3e5f1fb83 [^] Fixes issue 35171: Support basic authentication in the default authentication manager Add basic authentication support to DefaultAuthenticationManager --- M src/org/openbravo/authentication/basic/DefaultAuthenticationManager.java --- |
(0094207) alostale (viewer) 2017-02-10 08:51 |
code reviewed + tested |
(0095168) hudsonbot (viewer) 2017-03-15 20:19 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/54e102bef53e [^] Maturity status: Test |
![]() |
|||
Date Modified | Username | Field | Change |
2017-02-06 17:24 | mtaal | New Issue | |
2017-02-06 17:24 | mtaal | Assigned To | => mtaal |
2017-02-06 17:24 | mtaal | OBNetwork customer | => No |
2017-02-06 17:24 | mtaal | Modules | => Core |
2017-02-06 17:24 | mtaal | Triggers an Emergency Pack | => No |
2017-02-06 17:25 | mtaal | Relationship added | blocks 0035140 |
2017-02-08 09:38 | hgbot | Checkin | |
2017-02-08 09:38 | hgbot | Note Added: 0094140 | |
2017-02-08 09:39 | mtaal | Review Assigned To | => alostale |
2017-02-08 09:42 | hgbot | Checkin | |
2017-02-08 09:42 | hgbot | Note Added: 0094141 | |
2017-02-08 09:42 | hgbot | Status | new => resolved |
2017-02-08 09:42 | hgbot | Resolution | open => fixed |
2017-02-08 09:42 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/200a7c792da248caf9ae395d117904c3e5f1fb83 [^] |
2017-02-08 09:43 | mtaal | Relationship replaced | has duplicate 0035140 |
2017-02-10 08:51 | alostale | Note Added: 0094207 | |
2017-02-10 08:51 | alostale | Status | resolved => closed |
2017-02-10 08:51 | alostale | Fixed in Version | => 3.0PR17Q2 |
2017-03-15 20:19 | hudsonbot | Checkin | |
2017-03-15 20:19 | hudsonbot | Note Added: 0095168 |
Copyright © 2000 - 2009 MantisBT Group |