[aerogear-dev] AeroDoc: Show/Send leads - Geo map not showing agents near-by ?

Karel Piwko kpiwko at redhat.com
Mon Feb 17 08:02:19 EST 2014


As @Secure is "here will be dragons" kind of thing, I've filled documentation
request at - https://issues.jboss.org/browse/AGSEC-176.

Karel

On Mon, 17 Feb 2014 09:12:25 +0100
Matthias Wessendorf <matzew at apache.org> wrote:

> On Mon, Feb 17, 2014 at 9:07 AM, Sebastien Blanc <scm.blanc at gmail.com>wrote:
> 
> >
> >
> >
> > On Mon, Feb 17, 2014 at 9:00 AM, Matthias Wessendorf
> > <matzew at apache.org>wrote:
> >
> >>
> >>
> >>
> >> On Mon, Feb 17, 2014 at 8:57 AM, Sebastien Blanc
> >> <scm.blanc at gmail.com>wrote:
> >>
> >>>
> >>>
> >>>
> >>> On Mon, Feb 17, 2014 at 8:53 AM, Matthias Wessendorf
> >>> <matzew at apache.org>wrote:
> >>>
> >>>>
> >>>>
> >>>>
> >>>> On Mon, Feb 17, 2014 at 8:47 AM, Sebastien Blanc
> >>>> <scm.blanc at gmail.com>wrote:
> >>>>
> >>>>>
> >>>>>
> >>>>>
> >>>>> On Mon, Feb 17, 2014 at 8:42 AM, Matthias Wessendorf <
> >>>>> matzew at apache.org> wrote:
> >>>>>
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>> On Mon, Feb 17, 2014 at 8:20 AM, Erik Jan de Wit
> >>>>>> <edewit at redhat.com>wrote:
> >>>>>>
> >>>>>>>
> >>>>>>> On 16 Feb,2014, at 19:06 , Matthias Wessendorf <matzew at apache.org>
> >>>>>>> wrote:
> >>>>>>>
> >>>>>>> Hi,
> >>>>>>>
> >>>>>>> I tried to play w/ the Geo support on AeroDoc, but there are some
> >>>>>>> issues.
> >>>>>>>
> >>>>>>> After I noticed that only 'john' (the admin) (see [1]) is able to
> >>>>>>> update the log/lat data, I was wondering about the next thing: Does
> >>>>>>> the map show where 'agents' are? (by lat/long?)
> >>>>>>>
> >>>>>>>
> >>>>>>> Didn't change the security setting, guess it was always the case
> >>>>>>> that only admins can update. If that is wrong we should change that.
> >>>>>>>
> >>>>>>
> >>>>>> Hrm, ok - perhaps Sebi knows more
> >>>>>>
> >>>>> Yeah, that is because of this
> >>>>> https://github.com/aerogear/aerogear-aerodoc-backend/blob/master/src/main/java/org/jboss/aerogear/aerodoc/rest/SaleAgentEndpoint.java#L59
> >>>>> We should just override the update method with @Secure("simple")
> >>>>>
> >>>>
> >>>> You might wanna see here:
> >>>>
> >>>> https://github.com/aerogear/aerogear-aerodoc-backend/blob/master/src/main/java/org/jboss/aerogear/aerodoc/rest/SaleAgentEndpoint.java#L115
> >>>>
> >>>
> >>> Doh ! I'm probably not enough wake up :)
> >>> Ok but then we should maybe flip the coin and protect the class with
> >>> "simple" and override the "admin" ones with "admin".
> >>>
> >>
> >> well, shouldn't it just work, w/ the override ? Let's first investigate
> >> if the source is really a bug, versus a wrong usage
> >>
> >
> > Abstractj should confirm, but the override is not working this way,
> > looking at
> > https://github.com/aerogear/aerogear-security/blob/master/src/main/java/org/jboss/aerogear/security/interceptor/SecurityInterceptor.java#L54
> > it first validates on class level before looking at the method and an
> > Exception will be thrown before it looks at the method annotation.
> >
> 
> I see - sounds like incorrect usage on our demo application, in that case
> 
> 
> >
> >
> >>
> >> -<
> >>
> >>
> >>>
> >>>
> >>>>
> >>>>
> >>>>
> >>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> I am not seeing an agent on my map:
> >>>>>>> http://people.apache.org/~matzew/geo-map.png
> >>>>>>>
> >>>>>>> That is not supported at the moment.
> >>>>>>>
> >>>>>>
> >>>>>> Too bad ;-) Would be a nice feature :-)
> >>>>>>
> >>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> My long/lat values are OUTSIDE of that given radius (how can I
> >>>>>>> change the radius), but my location is INSIDE of the visible map.
> >>>>>>>
> >>>>>>>
> >>>>>>> You click the radius and drag the handle (little circle on the
> >>>>>>> bottom right) to increase the radius.
> >>>>>>>
> >>>>>>
> >>>>>> Ahhh!!!!! :-) I noticed that yesterday night :-) Almost an easter egg
> >>>>>> :-) pretty good hidden :) But I guess that's due to the map we use,
> >>>>>> right ?
> >>>>>>
> >>>>>
> >>>>> Yeah, the handle is not very visible in the first place, Erik, could
> >>>>> we change the style of this handle (more contrast, other color?)
> >>>>>
> >>>>>>
> >>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> Since a map is shown on that 'send leads' page, I was hoping to
> >>>>>>> actually see a mark for my location. Or is this just not supported,
> >>>>>>> atm ?
> >>>>>>>
> >>>>>>>
> >>>>>>> right not supported, the idea is that you search sales agents that
> >>>>>>> are within the radius and send them the lead.
> >>>>>>>
> >>>>>>
> >>>>>> Not sure that works :-)
> >>>>>>
> >>>>>> I added a lead in San Francisco, and had my agent use my coordinates.
> >>>>>> I updated the radius (increased by a few meters), than I think the
> >>>>>> search results updated, but john (location data: Germany, Emsdetten)
> >>>>>> was showing up :-)
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> Greetings,
> >>>>>>> Matthias
> >>>>>>>
> >>>>>>>
> >>>>>>> [1] https://issues.jboss.org/browse/AGPUSH-535
> >>>>>>>
> >>>>>>> --
> >>>>>>> Matthias Wessendorf
> >>>>>>>
> >>>>>>> blog: http://matthiaswessendorf.wordpress.com/
> >>>>>>> sessions: http://www.slideshare.net/mwessendorf
> >>>>>>> twitter: http://twitter.com/mwessendorf
> >>>>>>> _______________________________________________
> >>>>>>> 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
> >>>>>>>
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>> --
> >>>>>> Matthias Wessendorf
> >>>>>>
> >>>>>> blog: http://matthiaswessendorf.wordpress.com/
> >>>>>> sessions: http://www.slideshare.net/mwessendorf
> >>>>>> twitter: http://twitter.com/mwessendorf
> >>>>>>
> >>>>>> _______________________________________________
> >>>>>> 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
> >>>>>
> >>>>
> >>>>
> >>>>
> >>>> --
> >>>> Matthias Wessendorf
> >>>>
> >>>> blog: http://matthiaswessendorf.wordpress.com/
> >>>> sessions: http://www.slideshare.net/mwessendorf
> >>>> twitter: http://twitter.com/mwessendorf
> >>>>
> >>>> _______________________________________________
> >>>> 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
> >>>
> >>
> >>
> >>
> >> --
> >> Matthias Wessendorf
> >>
> >> blog: http://matthiaswessendorf.wordpress.com/
> >> sessions: http://www.slideshare.net/mwessendorf
> >> twitter: http://twitter.com/mwessendorf
> >>
> >> _______________________________________________
> >> 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
> >
> 
> 
> 



More information about the aerogear-dev mailing list