[aerogear-dev] Few suggestions about push quickstarts

Lucas Holmquist lholmqui at redhat.com
Fri Jun 28 07:39:46 EDT 2013


On Jun 27, 2013, at 5:33 PM, Kris Borchers <kborcher at redhat.com> wrote:

> 
> 
> On Jun 27, 2013, at 16:15, Jay Balunas <jbalunas at redhat.com> wrote:
> 
>> 
>> On Jun 27, 2013, at 12:34 PM, Sebastien Blanc wrote:
>> 
>>> 
>>> 
>>> 
>>> On Thu, Jun 27, 2013 at 6:17 PM, Bruno Oliveira <bruno at abstractj.org> wrote:
>>> Good morning, today I was looking at the quickstart demo for push and
>>> would like to make some considerations and see what do you guys think.
>>> In this way we can file jiras to move forward.
>>> 
>>> - The quickstart make use of AeroGear Controller. IMO we should move to
>>> Resteasy
>>> 
>>> +1 , I have a local brach with resteasy, will be pushed tomorrow ;) 
>>> - Code formatting, do we have a template for it? I don't want to mess up
>>> with the project.
>>> Yeah, sorry, that is also my fault, is  https://github.com/aerogear/ide-config still up to date and shall I use this one ? 
>> 
>> Unfortunately I think that likely needs to be reviewed and updated, although it is as least a starting point.  
>> 
>> What are others using now?  Their own?  Other communities?
> 
> I pretty much follow idiomaticjs. Also, ST2 has plugins that I use to prevent white space at end of lines and lint code as I type plus, it auto detects the tab size and spaces vs tabs for indentation and sets the editor for me. 

yea, ST2!!


>> 
>>> 
>>> - Something that brought to my attention, after discuss with Passos some
>>> issues on Android is when you send: curl -v -b cookies.txt -c
>>> cookies.txt -H "Accept: application/json" -H "Content-type:
>>> application/json" -X POST -d '{"loginName": "john", "password":"123"}'
>>> http://localhost:8080/prodoctor/login
>>> 
>>> The HTTP response is:
>>> 
>>> {"id":"8a7d9bfd-6adc-475a-9b90-407efb6bcae5","enabled":true,"createdDate":1372349593981,"expirationDate":null,"partition":null,"loginName":"john","firstName":null,"lastName":null,"email":null,"status":"PTO","password":"123","location":"New
>>> York"}
>>> 
>>> Attributes like expirationDate, partition and mailing password should
>>> never be sent back. For more details please take a look at how aerogear
>>> controller demo handle it
>>> https://github.com/aerogear/aerogear-controller-demo/blob/master/src/main/java/org/jboss/aerogear/controller/demo/Login.java#L48.
>>> 
>>> Behind the scenes PicketLink already encrypts the passwords on AGSec,
>>> but I can't do so much if they're sent back through the network. Thoughts?
>>> 
>>> Yes I have to filter the answer like you did in controller, thanks for pointing that out. 
>>> 
>>> --
>>> abstractj
>>> 
>>> _______________________________________________
>>> aerogear-dev mailing list
>>> aerogear-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>> 
>>> _______________________________________________
>>> aerogear-dev mailing list
>>> aerogear-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>> 
>> _______________________________________________
>> aerogear-dev mailing list
>> aerogear-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20130628/4d91d182/attachment.html 


More information about the aerogear-dev mailing list