Provide set/get methods for webPath used to set alternate web context…

[deleolajide]

… other than root

See https://github.com/guusdk/httpfileuploadcomponent/issues/15

[guusdk]

This is great, thanks!

What you've added so far, is a way to change the URL that is 'announced' to the outside world. Your changes do not affect the actual URL on which the service is being made available.

I have three suggestions for improvements:

• I think 'context root' is a better identifier than 'web path'.
• Users should be able to configure the new option you added via the launcher. Please extend the Launcher class with a new option announcedContextRoot, and add its value to the initialization here: https://github.com/guusdk/httpfileuploadcomponent/blob/4fb202d96b9c14d7bb7a2cee7f6da34454297a3e/src/main/java/nl/goodbytes/xmpp/xep0363/Launcher.java#L306-L308
• Apart from the "announced" context root, we should allow for the context root that is used by the embedded server to be configured (it's currently hardcoded to /). Please add another option (contextRoot) for this, and apply that option here: https://github.com/guusdk/httpfileuploadcomponent/blob/4fb202d96b9c14d7bb7a2cee7f6da34454297a3e/src/main/java/nl/goodbytes/xmpp/xep0363/Launcher.java#L329-L332

The default value for contextRoot should be: \ The default value for announcedContextRoot should be the value of contextRoot.

[guusdk]

Oh and please update the readme's with the new options. See why it's taking me so long to update this thing? :)

[guusdk]

15 has now been fixed with fd253d19d8578173637fb6bb82b3529acdb91dfa which incorporates your changes plus the feedback that I had.