Skip to content

DOC-14046: encryption at rest eventing#221

Open
Pallavi-Janardhan wants to merge 5 commits into
DOC-13938-encryption-at-restfrom
DOC-14046-encryption-at-rest-eventing
Open

DOC-14046: encryption at rest eventing#221
Pallavi-Janardhan wants to merge 5 commits into
DOC-13938-encryption-at-restfrom
DOC-14046-encryption-at-rest-eventing

Conversation

@Pallavi-Janardhan

@Pallavi-Janardhan Pallavi-Janardhan commented Jun 19, 2026

Copy link
Copy Markdown

Made the necessary changes as suggested in the Doc issues:

Docs preview

description: Returns the most recent application log messages for the specified function.
description: |-
Returns the most recent application log messages for the specified function.
Use this endpoint to view Eventing application logs when encryption at REST is enabled.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Can we reference encryption at REST from docs to give more context to the user? We can do it as a future item if it's not present in docs yet.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sure — see comment in my review

@simon-dew simon-dew changed the title Doc 14046 encryption at rest eventing DOC-14046: encryption at rest eventing Jun 25, 2026

@simon-dew simon-dew left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Some minor changes; also added link to eventing at rest concept page

description: Returns the most recent application log messages for the specified function.
description: |-
Returns the most recent application log messages for the specified function.
Use this endpoint to view Eventing application logs when encryption at REST is enabled.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Encryption at rest is not about REST (Representational State Transfer) so should not be all caps

Add link to concept doc for encryption at rest

Suggested change
Use this endpoint to view Eventing application logs when encryption at REST is enabled.
Use this endpoint to view Eventing application logs when encryption at rest is enabled.
For more information, see [Native Encryption at Rest](../learn/security/native-encryption-at-rest-overview.html).

Comment on lines +8 to +13
{
"target": "components.schemas.settings_schema.properties.enable_applog_rotation",
"update": {
"description": "Enable rotation for this handler's log() message files.\n\nDo not disable log rotation when encryption at rest is active, as this could compromise security or lead to unexpected behavior."
}
},

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just tidying up the indentation to match the rest of the file

Suggested change
{
"target": "components.schemas.settings_schema.properties.enable_applog_rotation",
"update": {
"description": "Enable rotation for this handler's log() message files.\n\nDo not disable log rotation when encryption at rest is active, as this could compromise security or lead to unexpected behavior."
}
},
{
"target": "components.schemas.settings_schema.properties.enable_applog_rotation",
"update": {
"description": "Enable rotation for this handler's log() message files.\n\nDo not disable log rotation when encryption at rest is active, as this could compromise security or lead to unexpected behavior."
}
},

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants