Skip to content

Conversation

@kennyj42
Copy link
Contributor

@kennyj42 kennyj42 commented Nov 5, 2024

This was incorrect, we look at the last seen value from gateway

Summary

Screenshots (optional)

Documentation checklist

  • The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

This was incorrect, we look at the last seen value from gateway
## What constitutes an authentication event

For Access, this is any Cloudflare Access authentication event, like a login to the [App Launcher](/cloudflare-one/applications/app-launcher/) or an application. For Gateway, this means any Cloudflare WARP authentication event, like enrolling a device to your ZT organization.
For Access, this is any Cloudflare Access authentication event, like a login to the [App Launcher](/cloudflare-one/applications/app-launcher/) or an application. For Gateway, this is if any devices associated have been seen in the specified period.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For Access, this is any Cloudflare Access authentication event, like a login to the [App Launcher](/cloudflare-one/applications/app-launcher/) or an application. For Gateway, this is if any devices associated have been seen in the specified period.
For Access, this is any Cloudflare Access authentication event, like a login to the [App Launcher](/cloudflare-one/applications/app-launcher/) or an application. For Gateway, this is if any devices associated with the user have been seen in the specified period.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the specified period?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The period is the setting in https://developers.cloudflare.com/cloudflare-one/identity/users/seat-management/#enable-seat-expiration.

However after looking at this page again I think we should leave the original statement. That is what constitutes an authentication event. However we consider any seat active for the purposes of seat expiration if they have any devices associated that have been seen. I'm not sure how we should rework the page for that.

Copy link
Contributor

@ranbel ranbel Nov 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

However we consider any seat active for the purposes of seat expiration if they have any devices associated that have been seen.

@maxvp perhaps this info could go into the Seat Expiration section:
https://github.com/cloudflare/cloudflare-docs/pull/18081/files#diff-50a1ac1c6b0bd5ec93cc24a705dc80e9bed38d044bc889dc39d41eba6bdb2ce1R60

@ranbel ranbel assigned maxvp and unassigned ranbel Nov 11, 2024
@ranbel ranbel requested a review from maxvp November 11, 2024 17:07
@ranbel
Copy link
Contributor

ranbel commented Nov 11, 2024

relates to #18081

@maxvp
Copy link
Contributor

maxvp commented Nov 19, 2024

@kennyj42 @jroyal I recently updated this page via #18081. Do the changes in this PR still apply?

@kennyj42
Copy link
Contributor Author

Yes they would. Your change still has the "or trigger a WARP enrollment" portion which isn't correct.

@maxvp maxvp enabled auto-merge (squash) November 19, 2024 21:15
@cloudflare-workers-and-pages
Copy link

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6ace9d2
Status: ✅  Deploy successful!
Preview URL: https://9f72069e.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://kennyj42-patch-3.cloudflare-docs-7ou.pages.dev

View logs

@github-actions
Copy link
Contributor

@maxvp maxvp merged commit 6e99f04 into production Nov 19, 2024
12 checks passed
@maxvp maxvp deleted the kennyj42-patch-3 branch November 19, 2024 21:25
harshil1712 pushed a commit that referenced this pull request Dec 3, 2024
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.

4 participants