Add runtime safety checks when publishing to external topics #1703
Comments
Open
This was referenced Jun 7, 2024
Closed
matt2e
added a commit
that referenced
this issue
Jun 11, 2024
Closes #1640 Checks for: - publishing directly to an external module's exported topic - directly calling an external module's verb, rather than using `ftl.Call(...)` - This already fails at runtime, but it's nice to have this as a compile error Both of these compile-time checks are not fool proof, any indirection will mean these checks don't catch it. But it is good to catch what we can. eg: ```go var extTopic = external.Topic extTopic.Publish(...) // compiler believes this is a local topic ``` Added a separate issue for this for pubsub as it may not be as high a priority: #1703
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Added compile time checks here: #1702
There are ways around the compile time checks, so we should also add runtime safety
The text was updated successfully, but these errors were encountered: