Skip to content

Extension validation config#2120

Merged
swaldmann merged 15 commits intomainfrom
extension-validation-config
Oct 15, 2025
Merged

Extension validation config#2120
swaldmann merged 15 commits intomainfrom
extension-validation-config

Conversation

@ecklie
Copy link
Contributor

@ecklie ecklie commented Sep 26, 2025

Separate section with some more details around extension allowlists

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Linting Errors

  • guides/multitenancy/mtxs.md:1119 Fenced code blocks should have a language specified

#### Restrict / Enable Annotations

The following annotations are blocked by default
```
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
```
```txt
@restrict
@requires
@readonly
@mandatory
@assert.*
@cds.persistence.*
@sql.append
@sql.prepend
@path
@impl
@cds.autoexpose
@cds.api.ignore
@odata.etag
@cds.query.limit
@cds.localized
@cds.valid.*
@cds.search
```

Please add a language tag. For plain text add txt as language tag.

@ecklie ecklie marked this pull request as ready for review September 30, 2025 12:32

The job and task status can take on the values `QUEUED`, `RUNNING`, `FINISHED` and `FAILED`.

### Extension Restriction
Copy link
Contributor

@swaldmann swaldmann Sep 30, 2025

Choose a reason for hiding this comment

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

Calling it "Extension Restrictions" above in L813 ;)

Suggested change
### Extension Restriction
### Extension Restrictions


### Extension Restriction

You can restrict what parts of the application model can be extended by the SaaS Customer.
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
You can restrict what parts of the application model can be extended by the SaaS Customer.
You can restrict what parts of the application model can be extended by the SaaS customer.

Comment on lines +1054 to +1055
> As soon as an `extension-allowlist` is present, extensions are forbidden, except those that are listed
in the `extension-allowlist`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Shorter:

Suggested change
> As soon as an `extension-allowlist` is present, extensions are forbidden, except those that are listed
in the `extension-allowlist`.
> If an `extension-allowlist` is defined, only the configured extensions are allowed.


#### Restrict Service Extensions

By adding services to the `extension-allowlist`, services are enabled for extensions by Saas Customers.
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
By adding services to the `extension-allowlist`, services are enabled for extensions by Saas Customers.
By adding services to the `extension-allowlist`, services are enabled for extensions by Saas customers.

#### Restrict Service Extensions

By adding services to the `extension-allowlist`, services are enabled for extensions by Saas Customers.
In addition, you can restrict the number of bound entities by setting a number for "new-entites".
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
In addition, you can restrict the number of bound entities by setting a number for "new-entites".
In addition, you can restrict the number of bound entities by setting a limit for "new-entites".

@swaldmann
Copy link
Contributor

swaldmann commented Sep 30, 2025

👍 just a few minor things

@swaldmann swaldmann merged commit a9eaffd into main Oct 15, 2025
7 checks passed
@swaldmann swaldmann deleted the extension-validation-config branch October 15, 2025 07:48
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.

2 participants