Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0021685 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | minor | always | 2012-09-19 09:57 | 2012-10-04 11:39 | |||
Reporter | iperdomo | View Status | public | |||||
Assigned To | AugustoMauch | |||||||
Priority | urgent | Resolution | no change required | Fixed in Version | ||||
Status | closed | Fix in branch | pi | Fixed in SCM revision | ||||
Projection | none | ETA | none | Target Version | 3.0MP17 | |||
OS | Linux 64 bit | Database | PostgreSQL | Java version | OpenJDK 1.6.0_24 | |||
OS Version | 3.5.3-1-ARCH | Database version | 9.1.4 | Ant version | 1.8.2 | |||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | ||||||||
OBNetwork customer | No | |||||||
Web browser | Apple Safari, Google Chrome, Internet Explorer, Mozilla Firefox | |||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0021685: A non authenticated Component needs to have the AdminMode context by default | |||||||
Description | The development in issue 0021508 adds the support for non authenticated Components. Before the authentication happens the only available OBContext is AdminMode, therefore it makes sense that the code executed during the Component generation should be wrapped in a AdminMode context. | |||||||
Steps To Reproduce | * Using a non-authenticated Component, try to access the database using DAL | |||||||
Proposed Solution | Wrap the Component instantiation and code generation in a AdminMode context | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() | |||||||
![]() |
||||||||
|
![]() |
|
(0052945) AugustoMauch (administrator) 2012-10-01 10:37 |
A patch for this issue has been attached. It will be applied to pi when it is not logically frozen for MP16. |
(0053025) iperdomo (viewer) 2012-10-03 20:01 |
@AugustoMauch: It is really need to modify HttpSecureAppServlet? There a dozens/hundreds of classes extending from this class. I think that you should take a look into BaseKernelServlet that is a subclass of this one and handles the requests to the "KernelServlet" in Openbravo 3. |
(0053026) iperdomo (viewer) 2012-10-03 21:41 |
Thinking a bit more, notice that only *some* Component generation needs to be wrapped in a AdminContext. This one are the ones with bypassAuthentication() equals true. The current associated patch is not OK. |
(0053028) AugustoMauch (administrator) 2012-10-04 11:39 |
The is no need to make any change, because the generation of components is already wrapped in an admin context, see [1]. [1] https://code.openbravo.com/erp/devel/pi/file/34211605322e/modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/ComponentGenerator.java#l62 [^] |
![]() |
|||
Date Modified | Username | Field | Change |
2012-09-19 09:57 | iperdomo | New Issue | |
2012-09-19 09:57 | iperdomo | Assigned To | => marvintm |
2012-09-19 09:57 | iperdomo | Web browser | => Apple Safari, Google Chrome, Internet Explorer, Mozilla Firefox |
2012-09-19 09:57 | iperdomo | Modules | => Core |
2012-09-19 09:57 | iperdomo | OBNetwork customer | => No |
2012-09-19 09:57 | iperdomo | Relationship added | related to 0021508 |
2012-09-25 11:32 | marvintm | Web browser | Apple Safari, Google Chrome, Internet Explorer, Mozilla Firefox => Apple Safari, Google Chrome, Internet Explorer, Mozilla Firefox |
2012-09-25 11:32 | marvintm | Target Version | 3.0MP16 => 3.0MP17 |
2012-09-27 11:10 | AugustoMauch | Assigned To | marvintm => AugustoMauch |
2012-09-27 11:10 | AugustoMauch | Status | new => scheduled |
2012-09-27 11:10 | AugustoMauch | fix_in_branch | => pi |
2012-09-27 15:08 | AugustoMauch | Assigned To | AugustoMauch => shankarb |
2012-10-01 09:43 | shankarb | Assigned To | shankarb => AugustoMauch |
2012-10-01 10:36 | AugustoMauch | File Added: issue21685.diff | |
2012-10-01 10:37 | AugustoMauch | Note Added: 0052945 | |
2012-10-03 20:01 | iperdomo | Note Added: 0053025 | |
2012-10-03 21:41 | iperdomo | Note Added: 0053026 | |
2012-10-04 11:39 | AugustoMauch | Note Added: 0053028 | |
2012-10-04 11:39 | AugustoMauch | Status | scheduled => closed |
2012-10-04 11:39 | AugustoMauch | Resolution | open => no change required |
Copyright © 2000 - 2009 MantisBT Group |