Re: [xwiki-devs] [Proposal] URL format for temporary resources

On 15 Apr 2016, at 10:30, Marius Dumitru Florea <

On Thu, Apr 14, 2016 at 7:46 PM, Vincent Massol &lt;vincent(a)massol.net&gt;

> >> On 14 Apr 2016, at 16:52, Marius Dumitru Florea < > mariusdumitru.florea(a)xwiki.com&gt; wrote: >> >> On Thu, Apr 14, 2016 at 5:43 PM, Vincent Massol &lt;vincent(a)massol.net&gt; > wrote: >> >>> Hi devs, >>> >>> I’m implementing http://jira.xwiki.org/browse/XWIKI-10375 ("Refactor > the >>> temporary resource concept inside the Resource module”) and I need to >>> define a URL format for the new “tmp” resource type. >>> >>> I’m proposing the following: >>> >>> >> >>> http://<server>/<context>/tmp/<module id>/<serialized owner document >>> reference>/<module-dependent resource path> >>> >> >> Serialized document reference uses backslash to escape special

WDYM? The dot is escaped in the space name with a backslash only when the space name is serialized as a reference, which is not the case for the standard wiki page URL /xwiki/bin/view/Space.With.Dot/Page.With.Dot Having a slash or a backslash in the space or page name is less common

having a dot ("Version 1.2"). And the user might be willing to accept

having a backslash in the page (or attachment's) name can cause security issues with Tomcat, but I doubt he will accept to avoid dots.

That’s not a blocking issue anyway since we can easily transform them

> other characters when we serialized and do the opposite when we parse

> URL. > >> This is based on the existing TemporaryResourceReference at: >>> >>> >

>>> >>> For example: >>> >>> http:// >>> >

>>> >>> Note that in this example from the officeviewer macro the > module-dependent >>> resource path consists in: >>> >> >> >>> - base64(name of office attachment + hashcode(parameters)) >>> >> >> See http://jira.xwiki.org/browse/XWIKI-11528 for the rationale behind > it. I >> was trying to avoid backslash (from the serialized attachment

It could but it's less common, especially because most Operating Systems are not very friendly with these characters when used in file or folder names. > > Note that I wasn’t sure why you you didn’t compute the base64 of both

> name of attachment + the parameters instead of having 2 directory levels > consisting in the base64 of the attachment name + the hashcode of the > parameters as different path segments. Need to check XWIKI-11528, maybe > it’s there. > > IMO we need to treat all path segments in the same way and convert slash > and backslash into some other characters. I’m not sure we need the

> solution. But anyway this is an implementation detail of the

> module and not really related to the discussion of the generic Temporary > URL format. > > Thanks > -Vincent > >> - generated image name from PPT >>> >>> In this case, the implementation would generate the following file: >>> >>> >>> >

_______________________________________________ devs mailing list devs(a)xwiki.org http://lists.xwiki.org/mailman/listinfo/devs