[AMAGENTS-267] not enforced IP processing broken Created: 14/Nov/16 Updated: 28/Aug/18 Resolved: 28/Aug/18 |
|
Status: | Closed |
Project: | OpenAM Agents |
Component/s: | Web Agents |
Affects Version/s: | 4.0.0, 4.0.1 |
Fix Version/s: | 5.0.0.0, 4.2.0.0 |
Type: | Bug | Priority: | Critical |
Reporter: | Bernhard Thalmayr | Assignee: | Mareks Malnacs |
Resolution: | Fixed | Votes: | 0 |
Labels: | 4p1-known-issue, QA-Agent5, release-notes, test-candidate | ||
Remaining Estimate: | 0h | ||
Time Spent: | 40m | ||
Original Estimate: | 0h | ||
Environment: |
Ubuntu 12.04.3 LTS Server version: Apache/2.2.22 (Ubuntu) ######################################################^M
|
Verified Version/s: | |
QA Assignee: | edwardb |
Cases: | 16700 |
Support Ticket IDs: | |
Epic Link: | Web Agent 4.1.1 Release |
Description |
Configure not enforce IP processing that matches the client IP from the incoming request excerpt from Apache http server access.log 192.168.56.1 - - [14/Nov/2016:10:27:35 +0100] "GET / HTTP/1.1" 302 618 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:49.0) Gecko/20100101 Firefox/49.0" excerpt from central agent profile com.sun.identity.agents.config.notenforced.ip[0]=192.168.56.1 Although the IP is not enforced the agent enforces the request and access is denied. excerpt from agent debug log ... 2016-11-14 10:38:12.463 +0100 DEBUG [0x7fe2791f1700:3762][source/config_xml.c:263] am_parse_config_xml() com.sun.identity.agents.config.notenforced.ip is set to 1 value(s) ... 2016-11-14 10:38:12.464 +0100 DEBUG [0x7fe2791f1700:3762][source/process.c:552] handle_not_enforced(): 2016-11-14 10:38:12.464 +0100 DEBUG [0x7fe2791f1700:3762][source/process.c:613] handle_not_enforced(): application logout url feature is not enabled 2016-11-14 10:38:12.464 +0100 DEBUG [0x7fe2791f1700:3762][source/process.c:665] handle_not_enforced(): attempting match with absolute access denied url http://ubuntu1204.test.xyz/403.html 2016-11-14 10:38:12.464 +0100 DEBUG [0x7fe2791f1700:3762][source/process.c:691] handle_not_enforced(): created normalised access denied url http://ubuntu1204.test.xyz:80/403.html ready for matching with http://ubuntu1204.test.xyz:80/ 2016-11-14 10:38:12.464 +0100 DEBUG [0x7fe2791f1700:3762][source/process.c:730] handle_not_enforced(): client ip address 192.168.56.1 does not match 192.168.56.1 2016-11-14 10:38:12.464 +0100 DEBUG [0x7fe2791f1700:3762][source/process.c:756] handle_not_enforced(): validating http://ubuntu1204.test.xyz:80/ 2016-11-14 10:38:12.464 +0100 DEBUG [0x7fe2791f1700:3762][source/process.c:765] handle_not_enforced(): trying not enforced pattern /403.html 2016-11-14 10:38:12.464 +0100 DEBUG [0x7fe2791f1700:3762][source/process.c:785] handle_not_enforced(): validating http://ubuntu1204.test.xyz:80/ ignoring query attributes 2016-11-14 10:38:12.464 +0100 DEBUG [0x7fe2791f1700:3762][source/process.c:874] handle_not_enforced(): extended not enforced url validation feature is not enabled 2016-11-14 10:38:12.464 +0100 DEBUG [0x7fe2791f1700:3762][source/process.c:877] handle_not_enforced(): http://ubuntu1204.test.xyz:80/ is enforced |
Comments |
Comment by Mareks Malnacs [ 14/Nov/16 ] |
Agent4 supports only either the CIDR specification: |
Comment by Charles Sparey [ 21/Nov/16 ] |
Either this is a case where the legacy functionality needs to be restored OR the docs need to be corrected / updated. |
Comment by Andy Hall [ 21/Nov/16 ] |
Mareks Malnacs What is the process for customers with 3.x agents and legacy not-enforced config, to move to 4.x? |
Comment by Mareks Malnacs [ 21/Nov/16 ] |
Single IP address rules should be changed to CIDR specification:
Wildcard rules are not supported in agent 4. But they can be changed to CIDR notation. Address range rule works as before. |
Comment by Andy Hall [ 21/Nov/16 ] |
We either need to write upgrade tasks to do this, or support the legacy format. |
Comment by edwardb [ 25/Oct/17 ] |
Functional Test added |
Comment by Charles Sparey [ 14/Aug/18 ] |
Changing versions from 4.1.x. |
Comment by Ľubomír Mlích [ 28/Aug/18 ] |
this works according to latest 4.x test, reopening to mark as verified |