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

allow declaring traci permission changes as transient #14709

Open
namdre opened this issue Apr 11, 2024 · 2 comments
Open

allow declaring traci permission changes as transient #14709

namdre opened this issue Apr 11, 2024 · 2 comments

Comments

@namdre
Copy link
Contributor

namdre commented Apr 11, 2024

so that they can be ignored, in the same way as rerouter-based permission changes.
To be used with #13494

@namdre
Copy link
Contributor Author

namdre commented Apr 11, 2024

Either: by adding an optional flag to all of the allow/disallow methods
(4 lane methods and 4 edge methods due to supporting allow/disallow with a string and a stringList)

Or: by setting something like a permission mode that influences subsequent calls to setAllowed and setDisallowed

@behrisch which would would you prefer?

@behrisch behrisch added this to the 1.21.0 milestone Apr 23, 2024
@behrisch
Copy link
Contributor

I would prefer the flag. Changing the function signature is not the worst thing here but we need to change the protocol. Nevertheless I think it is preferable to have it as stateless as possible.

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

No branches or pull requests

2 participants