Re: Sling Model and Request Parameters

2017-01-19 Thread Christophe Jelger
Parameters-tt4065648.html#a4065663 -Original Message- From: Christophe Jelger [mailto:jel...@adobe.com] Sent: Tuesday, January 17, 2017 5:35 AM To: users@sling.apache.org Subject: Re: Sling Model and Request Parameters

Re: Sling Model and Request Parameters

2017-01-19 Thread Christophe Jelger
ling.73963.n3.nabble.com/Sling-Model-and-Request-Parameters-tt4065648.html#a4065663 -Original Message- From: Christophe Jelger [mailto:jel...@adobe.com] Sent: Tuesday, January 17, 2017 5:35 AM To: users@sling.apache.org Subject: Re: Sling Model and Request

RE: Sling Model and Request Parameters

2017-01-18 Thread Jason Bailey
#a4065663 -Original Message- From: Christophe Jelger [mailto:jel...@adobe.com] Sent: Tuesday, January 17, 2017 5:35 AM To: users@sling.apache.org Subject: Re: Sling Model and Request Parameters Hi everyone, There was a discussion back on Oct 11, 2016 about an extra injector+annotation to

Re: Sling Model and Request Parameters

2017-01-17 Thread Christophe Jelger
Hi everyone, There was a discussion back on Oct 11, 2016 about an extra injector+annotation to inject request parameters into Sling models. I couldn’t find any JIRA issue about that, and I haven’t seen anything like that in the code so I don’t think this is currently in progress. I just joined

RE: Sling Model and Request Parameters

2016-10-11 Thread Stefan Seifert
>Von: Daniel Klco [mailto:dk...@apache.org] >Gesendet: Montag, 10. Oktober 2016 16:42 >An: users@sling.apache.org >Betreff: Re: Sling Model and Request Parameters > >Agreed, it'd be nice to have, but request parameters cannot be included in >the default injectors list for th

Re: Sling Model and Request Parameters

2016-10-10 Thread Daniel Klco
Agreed, it'd be nice to have, but request parameters cannot be included in the default injectors list for the reason which Stefan lists. Would it make sense to use the @Source annotation and add support for injectors which are not considered by default? On Fri, Oct 7, 2016 at 8:40 AM, Stefan Seife

RE: Sling Model and Request Parameters

2016-10-07 Thread Stefan Seifert
hello christoph. i currently see no hard reason against it, and implementing such an injector+additional annotation would be easy. it opens a bit more "attack surface" because potentially unsafe data from outside is directly injected in your model (perhaps without you noticing it when you use