Rudder

I'm digging out this old ticket.
The goal of this property is to allow for url rewrite.
With localhost, it doesn't have much meaning. If you access directly to the IP adress of the machine, you shouldn't have any problem either.

But if you want to specify an URL on with to talk to the Web Interface, then it is mandatory

base.url is NOT mandatory, as said in #4
If it's not filled, then all resources will be searched in the relative path to where you are

for now to configure your base url (rudder url looks like : http://server:port*/baseurl*/url)

you have to modify /opt/rudder/jetty7/contexts/rudder.xml and change content of line (line 5 in default file)

        <Set name="contextPath">/XXX</Set>

then modify /etc/apache2/sites-enabled/rudder-default and change the XXX with what you put in previous file

 
       # Rudder webapp
       RewriteEngine on
       RewriteRule   ^/$ /XXX [R]

       ProxyPass         "/XXX" "http://localhost:8080/XXX" 
       ProxyPassReverse  "/XXX" "http://localhost:8080/XXX" 
       ProxyRequests     Off

That should allow you to access rudder with http://server:port/XXX

The technical review for the scala/html/js part is ok (even in Vincent sneaked in others corrections in 64be6907081de3722c656f1dec198f8b5b15fbbf)

Some more question on the HTML/CCS part:

- I think there is a missing "/" in rudder-web/src/main/webapp/secure/nodeManager/searchNodes.html line 25
- the special instruction for IE < 7 in common-layout.html, line 52, does not have an URL starting with "/"
- "http://localhost:8080/images/favicon.[png, ico]" in rudder-loading.html smell fishy
- I don't understand some modifications in 64be6907081de3722c656f1dec198f8b5b15fbb
- only the first witch add a "/" seems related to this bug, all other (adding .value, a case in pattern matching, removing css modifier ":first" for table) seems to belong to an other (set of) ticket. Did something be mixed in that commit ?

François ARMAND wrote:

Some more question on the HTML/CCS part:

- I think there is a missing "/" in rudder-web/src/main/webapp/secure/nodeManager/searchNodes.html line 25
- the special instruction for IE < 7 in common-layout.html, line 52, does not have an URL starting with "/"
- "http://localhost:8080/images/favicon.[png, ico]" in rudder-loading.html smell fishy

I corrected those in 52aa095f

- I don't understand some modifications in 64be6907081de3722c656f1dec198f8b5b15fbb
- only the first witch add a "/" seems related to this bug, all other (adding .value, a case in pattern matching, removing css modifier ":first" for table) seems to belong to an other (set of) ticket. Did something be mixed in that commit ?

I mixed some corrections more related to #2838 (some .value fixes some links (remove nodeId) or some display, pattern match removes a warning, the first was useless))

Do i have to revert that commit and make two of them ?

In rudder-server-root postinst script, I use base.url from rudder-web.properties in order to add an hostname in the inventory of the Rudder server (#2745 #2778).

• Use hostname -f but we can't rely on it to be used by Rudder
• Search for root server hostname in the LDAP with slapcat

I suppose the second choice is the better. Jon, do you agree ?

Nicolas PERRON wrote:

In rudder-server-root postinst script, I use base.url from rudder-web.properties in order to add an hostname in the inventory of the Rudder server (#2745 #2778).

With this attribute removed, a new solution should be found to know the hostname of the machine. I see two solutions:

• Use hostname -f but we can't rely on it to be used by Rudder
• Search for root server hostname in the LDAP with slapcat

I suppose the second choice is the better. Jon, do you agree ?

The fixes for these issues have been changed then this question doesn't need to be addressed anymore.

I have asked on the wsrong issue (#2745):

Another question:

In rudder-init.sh script, a part is made for set the server hostname. This hostname initially is used for:

• Set the base.url which permit to access Rudder WebUI
• Set the nodeHostname in LDAP (indeed, replace %POLICY_SERVER_HOSTNAME%)

The first action is not important and we can't stop to do it, but it's not the same for the second one. Should we let rudder-init.sh to insert into LDAP %POLICY_SERVER_HOSTNAME% ? At least, the nodeHostname will be fixed with the new inventory send to Rudder.

The answer is simple: The hostname should continue to be ask for the moment.

Jonathan CLARKE wrote:

This commit looks good, but I have a question: how will Rudder behave if "base.url" contains the setting we used to have in it? ie, "http://rudder.mycompany.com/rudder/" or something similar? Will links break, or is this supported, or just plain ignored?

I've tried to set some broken links in rudder-web.properties and it seems that base.url isn't used anymore.

base.url is not used anymore in rudder, I kept all the services and the property in the code, but it is never used in any part of the webapp.

I kept the possibility to use it in the webapp, and to use that property, but I removed every usage of it.

So you could set any value for it, it has no impact on the webapp.

I think we should keep the property, but commented, to keep a trace of the base.url used before migration.

We should at least indicate that the property is now useless in rudder