Skip to content

Comments

fix: notauthority da mode urpc sending to self#3000

Merged
NoelStephensUnity merged 6 commits intodevelop-2.0.0from
fix/da-mode-urpc-sending-to-self
Aug 6, 2024
Merged

fix: notauthority da mode urpc sending to self#3000
NoelStephensUnity merged 6 commits intodevelop-2.0.0from
fix/da-mode-urpc-sending-to-self

Conversation

@NoelStephensUnity
Copy link
Member

@NoelStephensUnity NoelStephensUnity commented Aug 4, 2024

This incudes some universal RPC fixes.

Changelog

  • Fixed: Issue where NotAuthorityTarget would include the service observer in the list of targets to send the RPC to as opposed to excluding the service observer as it should.
  • Fixed: Issue where ProxyRpcTargetGroup could attempt to send a message if there were no targets to send to.

Testing and Documentation

  • Includes integration test RpcProxyMessageTesting.
  • No documentation changes or additions were necessary.

NotAuthority should not include the service observer.
ProxyRpcTargetGroup should not send anything if there is no one to send to.
Adding comments about why this was changed
@NoelStephensUnity NoelStephensUnity changed the title fix: notauthorityda mode urpc sending to self fix: notauthority da mode urpc sending to self Aug 5, 2024
adding partial validation of this fix where it still has to be manually tested with a service connection.
updating the changelog entry.
@NoelStephensUnity NoelStephensUnity marked this pull request as ready for review August 6, 2024 00:07
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner August 6, 2024 00:07
@NoelStephensUnity NoelStephensUnity merged commit 0c9ac4e into develop-2.0.0 Aug 6, 2024
@NoelStephensUnity NoelStephensUnity deleted the fix/da-mode-urpc-sending-to-self branch August 6, 2024 03:06
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.

1 participant