Re: [webfinger] feedback for multiple "rel"

[Dick Hardt <[e-mail]>]

FYI: multiple query string params with the same name are turned into arrays or parsed into arrays in node.js

the following script:

var querystring = require('querystring')
console.log( querystring.parse( 'foo=bar&rel=token1&rel=token2' ) )
console.log( querystring.stringify( {'foo':'bar','rel':['token1','token2']} ) )

outputs:

{ foo: 'bar', rel: [ 'token1', 'token2' ] }
foo=bar&rel=token1&rel=token2

Perhaps Ruby devs should just move to the next cool language? :-)
Seriously though, what does the query string parser do in Ruby?

-- Dick

On Dec 22, 2012, at 5:30 PM, "Paul E. Jones" <[e-mail]> wrote:

> That depends on where they’re used.  If I tell you the “rel” value is “http://example.com/first one”, I don’t escape that as I write it.  Or are you saying there is text somewhere that already makes that, as written, illegal?
>  
> I’m quite favorable to a space-separated list of rel values in a single rel parameter, too.  I was concerned before that spaces would present an issue, but figured we could either we make spaces illegal in rel values (which we can do since those are fabricated things!) or we double-escape.  My preference is a single escape and a single rel parameter and spaces are illegal in the “rel” values themselves.
>  
> But, the forces that be pushed pretty hard to change from &rel=token1%20token to &rel=token1&rel=token2.  At the end of the day, it makes no difference to me since I just need to know which way to deal with it.
>  
> So Ruby can really only support only a single parameter with a given name?  It will not collect those into an array or anything?
>  
> Paul
>  
> From: James M Snell [mailto:[e-mail]] 
> Sent: Saturday, December 22, 2012 7:51 PM
> To: Paul E. Jones
> Cc: nov matake; Melvin Carvalho; [e-mail]
> Subject: Re: [webfinger] feedback for multiple "rel"
>  
> Spaces in rel values are already illegal.. specifically, spaces are disallowed in registered link relations and non-escaped spaces are disallowed in absolute IRI's.. A comma delimited list of rel's is better than multiple individual parameters.. even with the potential need for double encoding.
>  
>  
>  
> 
> On Sat, Dec 22, 2012 at 4:00 PM, Paul E. Jones <[e-mail]> wrote:
> If we made it a requirement that “rel” values have no spaces (which I would argue is a damn good thing for many reasons) then we would not have to double-encode.
>  
> Paul
>  
> From: [e-mail] [mailto:[e-mail]] On Behalf Of nov matake
> Sent: Saturday, December 22, 2012 11:12 AM
> To: Melvin Carvalho
> 
> Cc: [e-mail]
> Subject: Re: [webfinger] feedback for multiple "rel"
>  
> BTW, why double encoding lead "problems with things like canonical urls and search engines"?
> Can you provide more details?
>  
> If 2 rel included, the response would be different than when only 1 rel given.
> So I feel those 2 are not the same, and can be indexed as 2 resources..
>  
> On 2012/12/23, at 0:44, Melvin Carvalho <[e-mail]> wrote:
>  
> 
>  
> 
> On 22 December 2012 05:48, nov matake <[e-mail]> wrote:
> Hi,
> 
> I have a comment for they way to specify multiple "rel" values.
> 
> As a ruby library developer, my main target is rails developers.
> Since rails can't handle multiple same query keys, developers will need to hack query params parser in rails middleware layer.
> I can easily imagine it'll be an annoying part to support webfinger in rails.
> 
> Is the multiple "rel" case can be a space-delimitered (or some other character) strings like multiple redirect_uri in OAuth2?
> Or any reason for putting multiple same keys in query parameters?
> 
> each time you delimit a list, you have to be able to escape the delimiter, which can be a pain and also leads to problems with things like canonical urls and search engines
> 
> can you use mod_rewrite?
>  
> 
> Cheers,
> 
> Nov Matake
> _______________________________________________
> webfinger mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/webfinger
>  
>  
> 
> _______________________________________________
> webfinger mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/webfinger
> 
>  
> _______________________________________________
> webfinger mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/webfinger