Skip to content
This repository has been archived by the owner on Jul 28, 2023. It is now read-only.

Serialize/De-serialize Command principal for ScheduledCommand #35

Open
yuanfeiw opened this issue May 16, 2015 · 1 comment
Open

Serialize/De-serialize Command principal for ScheduledCommand #35

yuanfeiw opened this issue May 16, 2015 · 1 comment

Comments

@yuanfeiw
Copy link

ScheduledCommand is NOT using the command principal, rather the thread's principal. This is likely caused by Command principal not serializable.

@jonsequitur
Copy link
Owner

After talking this through with a few people I think that it's best not to serialize the original principal. This has a risk of leaking secrets, such as claims. And the command scheduler delivering commands should be considered a different actor in the system, e.g. a service principal. This implies that scheduling a command requires an authz check, which doesn't currently exist. Or, if the scheduled command is intended to persist permissions, they will need to be specific to the authz approach being used by the user of Its.Cqrs, and encrypted, e.g. JWT.

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

No branches or pull requests

2 participants