On 12/05/09 21:42, Shawn Walker wrote:
Padraig O'Briain wrote:
Here is an updated webrev
http://cr.opensolaris.org/~padraig/ips-12962-v1/
It does not make any changes to the logic but does make some layout
changes:
1) Change heading in Manage Publishers dialog
2) Add icons to Up and Down buttons
3) Remove origins expander in Modify Publisher dialog
4) Some minor layout changes in Manage Publishers and Modify
Publisher dialog.
This is beautiful. This is the absolute best the manage publishers
stuff has ever looked.
However, I did notice that it allowed me to add SSL information to one
of my publishers despite one of my origins being http.
Otherwise, this is great!
Cheers,
I need to clarify what validation ought to be done here. Some questions.
Are we allowed to mix http and https in origins and mirrors or must they
all be the same type as the URI for the selected repository for the
publisher?
Your comment states that if one of the origins are http then SSL
information should not be specified. This suggest that one can mix
http and https in origins.
The code which deals with adding SSL information is __update_ssl_creds.
This code ignores the specified the SSL key and cert if the URI for the
selected repository for the publisher is http. Is this the correct behavior?
Should we prevent the specification of SSL key and cert if the URI for
selected repository for the publisher is http?
Padraig
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
