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

Zope allows ids to start with `@@` but cannot publish corresponding objects ... #591

Closed
d-maurer opened this issue May 8, 2019 · 3 comments

Comments

@d-maurer
Copy link
Contributor

commented May 8, 2019

... because on traversal, it interprets the @@ as view indicator.

Same problem with other id structures which can be misinterpreted as view indicators, such
as ++namespace++name.

We likely should forbid ids which could be mistaken as view indicators.
For publishing itself, we could use (url-) quoting to distinguish the use of @ as part of an id from that as view indicator. However, this would not fit well with programmatic traversal.

@d-maurer d-maurer added the bug label May 8, 2019

@d-maurer

This comment has been minimized.

Copy link
Contributor Author

commented May 8, 2019

An alternative to forbidding affected ids could be to specify that one interpretation has priority over the other interpretation, e.g. first try to interpret a name as a view and if this fails try to interpret it as an id. Such an approach is already used for names not starting with @ or +.

@dataflake

This comment has been minimized.

Copy link
Member

commented May 8, 2019

My vote would be for forbidding IDs that clash with other functionality like those starting with @@ or ++. That's much easier to explain than the situation where people get confused because they created an object with such an "illegal" name but can't get to it.

@dataflake dataflake self-assigned this May 9, 2019

@dataflake dataflake added this to To do in Zope 4 final release via automation May 9, 2019

@dataflake dataflake added this to the 4.0 final milestone May 9, 2019

@dataflake

This comment has been minimized.

Copy link
Member

commented May 9, 2019

Fixed in fe0bc43

@dataflake dataflake closed this May 9, 2019

Zope 4 final release automation moved this from To do to Done May 9, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
2 participants
You can’t perform that action at this time.