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 | ||||||||
Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|||||||||
|
|||||||||
Notes |
|
|
(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 |
Issue History |
|||
| 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 |