Bug 1191071
Summary: | Permission error on proxy details page and connection tab for proxied hosts missing in spacewalk nightly/2.3 | ||
---|---|---|---|
Product: | [Community] Spacewalk | Reporter: | Patrick Hurrelmann <redhat> |
Component: | Proxy Server | Assignee: | Stephen Herr <sherr> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Red Hat Satellite QA List <satqe-list> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 2.2 | ||
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | spacewalk-java-2.3.146-1 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2015-04-14 19:03:51 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 1207293 |
Description
Patrick Hurrelmann
2015-02-10 12:34:33 UTC
I ported the acls incorrectly from perl. In the process of fixing. I believe I have fixed things so that the behavior of those pages should be the same as it was in Spacewalk 2.2, namely: Connections.do is shown as an option any time the org has at least one proxy. Proxy.do is shown as an option (and works) if the system is a proxy or could become one (the latter condition requires channel family entitlements that probably will only be true for Satellites). If you want to try out the newest spacewalk-java and see if that matches your expectations that would be great. Committing to Spacewalk master: a5a50d7c47488173b1ce9210c18de79dd1f38533 Thanks for that :) Most cases are fixed in spacewalk-java-2.3.142. "Connection" tab and "Proxy" tab are working, but the "Systems Using Proxy" on the "Proxy" tab is still not working. Probably the same fix as done for proxy.jsp is needed, but I was unable to build and test spacewalk-java due to missing dependencies :/ Maybe the following patch is enough, but as written above I could not build and test. --- code/webapp/WEB-INF/struts-config.xml.orig 2015-02-18 23:36:35.000000000 +0100 +++ code/webapp/WEB-INF/struts-config.xml 2015-02-19 09:45:05.525410627 +0100 @@ -2920,7 +2920,8 @@ input="/WEB-INF/pages/systems/sdc/proxyclients.jsp" type="com.redhat.rhn.frontend.action.systems.sdc.ProxyClientsAction" className="com.redhat.rhn.frontend.struts.RhnActionMapping"> - <set-property property="acls" value="org_channel_family(rhn-proxy)"/> + <set-property property="mixins" value="com.redhat.rhn.common.security.acl.SystemAclHandler"/> + <set-property property="acls" value="user_role(org_admin); system_feature(ftr_proxy_capable) or system_is_proxy(); org_channel_family(rhn-proxy) or system_is_proxy(); child_channel_candidate(rhn-proxy) or system_is_proxy(); org_entitlement(rhn_provisioning); not system_is_satellite(); system_feature(ftr_kickstart) or system_feature(ftr_snapshotting)"/> <forward name="default" path="/WEB-INF/pages/systems/sdc/proxyclients.jsp" /> </action> Ah, thanks I missed that one. It's similar, but that page should only displayed if the system is currently a proxy, so the acls are actually quite a bit simpler. Committing to Spacewalk master (under your name): cebe095e800dbdfd3a8573c374dfbdd0bd3a865a I can confirm the fix. No permission errors any longer after update to spacewalk-java > 2.3.146. Thank you very much :) Moving bugs to ON_QA as we move to release Spacewalk 2.3 Spacewalk 2.3 has been released. See https://fedorahosted.org/spacewalk/wiki/ReleaseNotes23 |