Re: [Freeipa-devel] [PATCH] 098 Forms based authentication UI

2012-03-02 Thread Petr Vobornik

On 03/02/2012 12:39 AM, Rob Crittenden wrote:

Petr Vobornik wrote:

Support for forms based authentication was added to UI.

It consist of:

1) new login page
Page url is [ipa server]/ipa/ui/login.html

Page contains a login form. For authentication it sends ajax request at
[ipa server]/session/json/login_password. If authentication is
successfull page is redirected to [ipa server]/ipa/ui if it fails from
whatever reason a message is shown.

2) new enhanced error dialog - authorization_dialog.

This dialog is displayed when user is not authorized to perform action -
usually when ticket and session expires.
It is a standard error dialog which shows kerberos ticket related error
message and newly offers (as a link) to use form based authentication.
If user click on the link, the dialog content and buttons switch to
login dialog which has same functionality as 'new login page'. User is
able to return back to the error message by clicking on a back button.

login.html uses same css styles as migration page - ipa-migration.css
was merged into ipa.css.

https://fedorahosted.org/freeipa/ticket/2450

Theoretically the login.html is not needed.
Sometime later we should come up with a method how to i18n static pages
and main page prior to authentication.


ACK. It looks like ipa.js in master and ipa-2-2 have diverged slightly,
I'll let you push this so you can make sure everything is ok.

rob


Pushed to master an ipa-2-2.

ipa.js is same in master and ipa-2-2. Maybe you were confused by the 
fact that when I was recently pushing 14 batches at once, I pushed two 
pairs of patches in different order. (Is it bad?) The final code is the 
same though.


--
Petr Vobornik

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Re: [Freeipa-devel] [PATCH] 098 Forms based authentication UI

2012-03-01 Thread Rob Crittenden

Petr Vobornik wrote:

Support for forms based authentication was added to UI.

It consist of:

1) new login page
Page url is [ipa server]/ipa/ui/login.html

Page contains a login form. For authentication it sends ajax request at
[ipa server]/session/json/login_password. If authentication is
successfull page is redirected to [ipa server]/ipa/ui if it fails from
whatever reason a message is shown.

2) new enhanced error dialog - authorization_dialog.

This dialog is displayed when user is not authorized to perform action -
usually when ticket and session expires.
It is a standard error dialog which shows kerberos ticket related error
message and newly offers (as a link) to use form based authentication.
If user click on the link, the dialog content and buttons switch to
login dialog which has same functionality as 'new login page'. User is
able to return back to the error message by clicking on a back button.

login.html uses same css styles as migration page - ipa-migration.css
was merged into ipa.css.

https://fedorahosted.org/freeipa/ticket/2450

Theoretically the login.html is not needed.
Sometime later we should come up with a method how to i18n static pages
and main page prior to authentication.


ACK. It looks like ipa.js in master and ipa-2-2 have diverged slightly, 
I'll let you push this so you can make sure everything is ok.


rob

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


[Freeipa-devel] [PATCH] 098 Forms based authentication UI

2012-02-29 Thread Petr Vobornik

Support for forms based authentication was added to UI.

It consist of:

1) new login page
Page url is [ipa server]/ipa/ui/login.html

Page contains a login form. For authentication it sends ajax request at 
[ipa server]/session/json/login_password. If authentication is 
successfull page is redirected to [ipa server]/ipa/ui if it fails from 
whatever reason a message is shown.


2) new enhanced error dialog - authorization_dialog.

This dialog is displayed when user is not authorized to perform action - 
usually when ticket and session expires.
It is a standard error dialog which shows kerberos ticket related error 
message and newly offers (as a link) to use form based authentication. 
If user click on the link, the dialog content and buttons switch to 
login dialog which has same functionality as 'new login page'. User is 
able to return back to the error message by clicking on a back button.


login.html uses same css styles as migration page - ipa-migration.css 
was merged into ipa.css.


https://fedorahosted.org/freeipa/ticket/2450

Theoretically the login.html is not needed.
Sometime later we should come up with a method how to i18n static pages 
and main page prior to authentication.

--
Petr Vobornik
From 17c4675f36c684563a259610a02e65c56e21aea6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Petr=20Voborn=C3=ADk?= pvobo...@redhat.com
Date: Mon, 27 Feb 2012 15:31:20 +0100
Subject: [PATCH] Forms based authentication UI

Support for forms based authentication was added to UI.

It consist of:

1) new login page
Page url is [ipa server]/ipa/ui/login.html

Page contains a login form. For authentication it sends ajax request at [ipa server]/session/json/login_password. If authentication is successfull page is redirected to [ipa server]/ipa/ui if it fails from whatever reason a message is shown.

2) new enhanced error dialog - authorization_dialog.

This dialog is displayed when user is not authorized to perform action - usually when ticket and session expires.
It is a standard error dialog which shows kerberos ticket related error message and newly offers (as a link) to use form based authentication. If user click on the link, the dialog content and buttons switch to login dialog which has same functionality as 'new login page'. User is able to return back to the error message by clicking on a back button.

login.html uses same css styles as migration page - ipa-migration.css was merged into ipa.css.

https://fedorahosted.org/freeipa/ticket/2450
---
 freeipa.spec.in |7 +-
 install/migration/Makefile.am   |1 -
 install/migration/error.html|1 -
 install/migration/index.html|1 -
 install/migration/invalid.html  |5 +-
 install/migration/ipa_migration.css |   96 
 install/ui/Makefile.am  |2 +
 install/ui/dialog.js|   39 --
 install/ui/field.js |6 +-
 install/ui/ipa.css  |   99 -
 install/ui/ipa.js   |  287 ++-
 install/ui/jsl.conf |1 +
 install/ui/login.html   |   51 ++
 install/ui/login.js |   76 +
 install/ui/test/data/ipa_init.json  |8 +-
 install/ui/widget.js|1 -
 ipalib/plugins/internal.py  |6 +
 17 files changed, 527 insertions(+), 160 deletions(-)
 delete mode 100644 install/migration/ipa_migration.css
 create mode 100644 install/ui/login.html
 create mode 100644 install/ui/login.js

diff --git a/freeipa.spec.in b/freeipa.spec.in
index 44e4828643da8ee4f4f715ffe4b7302e6c0b14f7..5047db938cbbd4728c90fa0d321a6bc8bb0e19ae 100644
--- a/freeipa.spec.in
+++ b/freeipa.spec.in
@@ -547,10 +547,10 @@ fi
 %{_usr}/share/ipa/migration/error.html
 %{_usr}/share/ipa/migration/index.html
 %{_usr}/share/ipa/migration/invalid.html
-%{_usr}/share/ipa/migration/ipa_migration.css
 %{_usr}/share/ipa/migration/migration.py*
 %dir %{_usr}/share/ipa/ui
 %{_usr}/share/ipa/ui/index.html
+%{_usr}/share/ipa/ui/login.html
 %{_usr}/share/ipa/ui/logout.html
 %{_usr}/share/ipa/ui/*.ico
 %{_usr}/share/ipa/ui/*.css
@@ -668,6 +668,11 @@ fi
 %ghost %attr(0644,root,apache) %config(noreplace) %{_sysconfdir}/ipa/ca.crt
 
 %changelog
+
+* Wed Feb 29 2012 Petr Vobornik pvobo...@redhat.com - 2.2.0-14
+- Add Web UI form based login page
+- Removed ipa_migration.css
+
 * Wed Feb 29 2012 Petr Vobornik pvobo...@redhat.com - 2.2.0-13
 - Add Web UI logout page
 
diff --git a/install/migration/Makefile.am b/install/migration/Makefile.am
index aa571364084872a88eaea2410506e1c432e747af..b90578015060c65be8e2d60bf486fd2606394873 100644
--- a/install/migration/Makefile.am
+++ b/install/migration/Makefile.am
@@ -5,7 +5,6 @@ app_DATA =  \
 	error.html			\
 	index.html			\
 	invalid.html			\
-	ipa_migration.css			\
 	migration.py			\
 	$(NULL)
 
diff --git a/install/migration/error.html b/install/migration/error.html
index