Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Abdul/UI timeout #5324

Closed
wants to merge 41 commits into from
Closed

Abdul/UI timeout #5324

wants to merge 41 commits into from

Conversation

iamazzeez
Copy link
Contributor

@iamazzeez iamazzeez commented Jul 8, 2021

Things Introduced.

  • Automate UI checks for user activities such as mouse movement, click, keyboard key press, scroll etc.
  • If there is no activity from user's side for the time specified for idle_timeout_minutes in config.toml then user will be explicitly logged out.
  • This feature can be enabled/disabled by enable_idle_timeout this property in config.toml
  • User kept multiple tabs/windows open but active on any one of the tab/window, to accommodate this use-case, such that user is not logged out as user is active. (https://github.com/pubkey/broadcast-channel) is used, It supports all browsers.

👟 How to Build and Test the Change\

  • Pull changes from branch abdul/ui_timeout
  • update session.toml to desired values.Screenshot 2021-07-15 at 9.29.45 AM.png

cd /src
rebuild components/automate-load-balance
rebuild components/automate-ui
rebuild components/automate-deployment
rebuild components/automate-cli

  • chef-automate config patch dev/session.toml

All PRs must tick these:

@netlify
Copy link

netlify bot commented Jul 8, 2021

👷 Deploy Preview for chef-automate processing.

🔨 Explore the source changes: 7388927

🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-automate/deploys/60efae4fd091d900076f6276

@kalroy kalroy marked this pull request as draft July 9, 2021 06:15
kalroy and others added 6 commits July 9, 2021 23:09
Signed-off-by: Kallol Roy <karoy@progress.com>
Signed-off-by: Abdul-az <aazeez@progress.com>
Signed-off-by: Abdul-az <aazeez@progress.com>
Signed-off-by: Abdul-az <aazeez@progress.com>
iamazzeez and others added 13 commits July 12, 2021 08:49
Signed-off-by: Abdul-az <aazeez@progress.com>
Signed-off-by: Kallol Roy <karoy@progress.com>
* Removed bintray dependecies from deployment

Signed-off-by: Kallol Roy <karoy@progress.com>

* New bundle creator

Signed-off-by: Kallol Roy <karoy@progress.com>

* Removed bin from test

Signed-off-by: Kallol Roy <karoy@progress.com>

* Removed date versioned hab from test

Signed-off-by: Kallol Roy <karoy@progress.com>

* Getting Binlink back

Signed-off-by: Kallol Roy <karoy@progress.com>

* Migrate away from Bintray for migration

Signed-off-by: Kallol Roy <karoy@progress.com>
* Added UI for delete part

Signed-off-by: chaitali-mane <cmane@progress.com>

* Added API integration

Signed-off-by: chaitali-mane <cmane@progress.com>

* Added test cases

Signed-off-by: chaitali-mane <cmane@progress.com>

* Updated API integration code

Signed-off-by: chaitali-mane <cmane@progress.com>

* minor change

Signed-off-by: chaitali-mane <cmane@progress.com>

* Minor change

Signed-off-by: chaitali-mane <cmane@progress.com>

* Minor change

Signed-off-by: chaitali-mane <cmane@progress.com>
Signed-off-by: Abdul-az <aazeez@progress.com>
Signed-off-by: Abdul-az <aazeez@progress.com>
* Enable HSTS policy for the routes which does not have the header set as of now.

Signed-off-by: Kallol Roy <karoy@progress.com>

* More security fixes

Signed-off-by: Kallol Roy <karoy@progress.com>

* Review changes are done

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

Co-authored-by: Venkatesh-rengasamy <vrengasa@progress.com>
* Telemetry Pref is restored while login

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning - remove unnecessary changes

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning - remove unnecessary changes

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Telemetry Preference issue

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Show telemetry warning message if preference is disabled

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleanup

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Restrict unnecessary api calls for segment write key

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* add static warning message with checkbox

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Update warning message and its alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleanup

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* anchor tag security issue in pipeline

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

* Telemetry Pref is restored while login

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning - remove unnecessary changes

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning - remove unnecessary changes

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Telemetry Preference issue

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Show telemetry warning message if preference is disabled

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleanup

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Restrict unnecessary api calls for segment write key

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* add static warning message with checkbox

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Update warning message and its alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleanup

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* anchor tag security issue in pipeline

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

* upgrade highlight.js version

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

* upgrade typedoc version for build error

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

* fix protactor failed case - telemetry

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>
Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>
Signed-off-by: Abdul-Az <iamazzeez@gmail.com>
Signed-off-by: Abdul-Az <iamazzeez@gmail.com>
Signed-off-by: Abdul-Az <iamazzeez@gmail.com>
@iamazzeez iamazzeez self-assigned this Jul 15, 2021
@sonarcloud
Copy link

sonarcloud bot commented Jul 15, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@iamazzeez iamazzeez changed the title Abdul/UI timeout Draft Abdul/UI timeout Jul 15, 2021
@iamazzeez iamazzeez marked this pull request as ready for review July 16, 2021 06:33
@kalroy kalroy marked this pull request as draft July 18, 2021 06:06
@kalroy
Copy link
Collaborator

kalroy commented Aug 2, 2021

Closing this as we are tracking this PR #5362

@kalroy kalroy closed this Aug 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants