#41: Switch fedocal to use flask_fas_openid ---------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: new Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Keywords: | ---------------------+------------------------- At the moment we are using flask_fas, we should switch to openid.
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: new Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: | Keywords: ----------------------+-------------------------
Comment (by lbrabec):
I attached patch proposal, package python-fedora-flask is needed.
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: new Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: | Keywords: ----------------------+-------------------------
Comment (by pingou):
Thanks for your patch and starting to look into this.
I should have mentioned that puiterwijk also started to work on this, his patch is at: http://puiterwijk.fedorapeople.org/0001-Update-fedocal-to-use- flask_fas_openid.patch
Regarding your work, the line 53, 54 are not needed (the APP.config addition). Your approach of auth_login is better though.
I'll have to check for the change in the template, as I basically don't want to have the submenu User if you're not login, just a 'Login' button in the menu and the submenu only if you are logging. Could you look into this? :)
PS: if you could submit your patch using git format-patch that would be awesome as it would give me the possibility to attribute you the work ;-)
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: new Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: | Keywords: ----------------------+-------------------------
Comment (by lbrabec):
Ok, I removed config lines. No submenu if user not logged (with little hack to temporary disable accordion). And also patch created with git format-patch :-)
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: new Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: | Keywords: ----------------------+-------------------------
Comment (by pingou):
Just by looking at it, it looks good :)
One tiny comment, please use the flask.ext construct: {{{ - from flask_fas_openid import FAS + from flask.ext.fas_openid import FAS }}}
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: new Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: | Keywords: ----------------------+-------------------------
Comment (by pingou):
Also, you have 3 lines with trailing space
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: new Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: | Keywords: ----------------------+-------------------------
Comment (by pingou):
in this line: {{{ <h4> <a class="toDisable" href={{ url_for("auth_login", next=request.url) + '' }}>FAS Log in</a> </h4> }}} Why the {{{+ ''}}} ?
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: new Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: | Keywords: ----------------------+-------------------------
Comment (by lbrabec):
Just relic of testing, I overlooked it..
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: new Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: | Keywords: ----------------------+-------------------------
Comment (by pingou):
Looking good :)
I'm not a big fan of the hack on the accordion, but right now I don't see another solution so this might just do.
Thanks for the work!
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: new Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: | Keywords: ----------------------+-------------------------
Comment (by pingou):
btw, in the javascript part .live has been deprecated for .on, but I can adjust this before committing/pushing.
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: closed Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: fixed | Keywords: ----------------------+------------------------- Changes (by pingou):
* resolution: => fixed * status: new => closed
Comment:
Well, this is now fixed and is the last item open for 0.2.0 :)
#41: Switch fedocal to use flask_fas_openid ----------------------+------------------------- Reporter: pingou | Owner: Type: defect | Status: closed Priority: major | Milestone: 0.2.0 Component: fedocal | Version: development Resolution: fixed | Keywords: ----------------------+-------------------------
Comment (by pingou):
Forgot to add, thanks for your work!
fedocal-devel@lists.fedorahosted.org