Comment 10 for bug 1606646

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/22863
Committed: http://github.org/Juniper/contrail-web-core/commit/d7fbde051679542970597cf680e4026403eeb1cf
Submitter: Zuul
Branch: master

commit d7fbde051679542970597cf680e4026403eeb1cf
Author: Biswajit Mandal <email address hidden>
Date: Thu Jul 21 15:25:30 2016 +0530

Closes-Bug: #1605163
This is side effect of the below fix:
https://github.com/Juniper/contrail-web-core/commit/f10620ede1f9f28858a9b418ec26093c57f00c41
Comuting data before Content-Length header assignement.

Change-Id: I100ab66856595688e97a74a54ce534fcdc201a36
(cherry picked from commit 15ba7405f1a000dcf3537cef7c2191f91a1c0fb5)

Closes-Bug: #1606646
Post node version 0.9.1, tls, https: validate server certificate by default.
So in case self signed certificate, UI will not work with https request to
openstack services.
https://github.com/mscdex/node-imap/issues/181#issuecomment-14781480
Fix: Added rejectUnauthorized flag.

In node post v0.11.14, we have custom checkServerIdentity in options to have
more control.

UT Results:
+++++++++++
contrail-web-controller:
------------------------
PhantomJS 1.9.8 (Mac OS X 10.9.5): Executed 376 of 376 SUCCESS (1 min 41.645 secs / 1 min 39.143 secs)
Done, without errors.

contrail-web-server-manager:
----------------------------
PhantomJS 1.9.8 (Mac OS X 10.9.5): Executed 89 of 89 SUCCESS (11.9 secs / 11.643 secs)
Done, without errors.

contrail-web-storage:
--------------------
PhantomJS 1.9.8 (Mac OS X 0.0.0): Executed 84 of 84 SUCCESS (20.301 secs / 18.748 secs)
Done, without errors.

Change-Id: Iec589a59b5ae153c3b6a57ad1e96a025c8a9280e
(cherry picked from commit 60c214946aa0ceb34e668a212a41036b693e63df)

Related-Bug: #1606775
Once we get logout request, immediately invalidate session authentication, and
clear _csrf, connect.sid cookie, do not wait for deleting the token.

UT Results:
+++++++++++
contrail-web-controller:
------------------------
PhantomJS 1.9.8 (Mac OS X 10.9.5): Executed 376 of 376 SUCCESS (1 min 41.645 secs / 1 min 39.143 secs)
Done, without errors.

contrail-web-server-manager:
----------------------------
PhantomJS 1.9.8 (Mac OS X 10.9.5): Executed 89 of 89 SUCCESS (11.9 secs / 11.643 secs)
Done, without errors.

contrail-web-storage:
--------------------
PhantomJS 1.9.8 (Mac OS X 0.0.0): Executed 84 of 84 SUCCESS (20.301 secs / 18.748 secs)
Done, without errors.

Change-Id: Ie11186071bb92f116b9472473170e19c58010adf
(cherry picked from commit b1603b99d7d14d812613c6874204f0c5e0ef5712)

Closes-Bug: #1584651
doDeepSort, align the keys according to the type of input object

Change-Id: I2c791ae2e45ed081fde9947508bf9ec2bdd25675
(cherry picked from commit c2913b7a05099679ab3c73686f3575ab04d822e1)

Closes-Bug: #1608383
In Microsoft EDGE Browser, response code 307, does not lead the browser to
redirect to certain URL, so sending 200 responseCode. (if userAgent
contains MSIE/Trident/Edge)

Change-Id: I4f971f66de19db915492884cbaf9ef0b09f05eb7
(cherry picked from commit 562c002dd70c48b430f8441f0cc3c7a756541113)

Closes-Bug: #1609425
In earlier releases we had /login and /vcenter/login, from R3.1
we have changed login to "/" and "/vcenter" only, so in case
user does "/login" or "/vcenter/login" then they should redirect
to "/" and "/vcenter" respectively.

Change-Id: I2424639aefd4e2a3291e492ec702096138da5d87
(cherry picked from commit e1c3185d6b9a226eb6f69fe6617a164a946c4096)

Closes-Bug: #1609425
Handled the case of /logout and /vcenter/logout as well, it used to come json
response, in stead of changing to login screen to come and changing the url as
per the orchestration system running.
In vcenter logout, after logout response from vcenter, do the next set of
operations.

Change-Id: I73d01daf4032fe7cc30d69e43bad80bfe176727d
(cherry picked from commit 30ed590163584b592aeb76555fdcaa255d5c70f1)