Skip to content

Feat/native flow galaxy message - #1

Merged
burhan-wesolves merged 2 commits into
developfrom
feat/native-flow-galaxy-message
Aug 1, 2026
Merged

Feat/native flow galaxy message#1
burhan-wesolves merged 2 commits into
developfrom
feat/native-flow-galaxy-message

Conversation

@burhan-wesolves

@burhan-wesolves burhan-wesolves commented Aug 1, 2026

Copy link
Copy Markdown
Owner

📋 Description

🔗 Related Issue

Closes #(issue_number)

🧪 Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🔧 Refactoring (no functional changes)
  • ⚡ Performance improvement
  • 🧹 Code cleanup
  • 🔒 Security fix

🧪 Testing

  • Manual testing completed
  • Functionality verified in development environment
  • No breaking changes introduced
  • Tested with different connection types (if applicable)

📸 Screenshots (if applicable)

image

✅ Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have manually tested my changes thoroughly
  • I have verified the changes work with different scenarios
  • Any dependent changes have been merged and published

📝 Additional Notes

basohangpur and others added 2 commits June 13, 2026 17:13
Adds a `flow` button type to sendButtons, mapping to the native_flow
`galaxy_message` interactive message. Sends flow_id, flow_token, flow_cta,
flow_action, flow_action_payload, mode through the existing native_flow
biz node. Flow buttons must travel alone (own template), mirroring PIX.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The CVE-2435 sanitizer (evolution-foundation#2549) stripped instanceId/instanceName from all
query params, including on the param-less GET /instance/fetchInstances
route. With instanceId dropped, the controller fell through to returning
ALL instances, so the manager UI (which reads data[0]) always opened the
first-created instance regardless of which card was clicked.

Only sanitize query/body on instance-scoped routes (those carrying a
:instanceName URL param, where the param is the authenticated identity).
Param-less routes keep their instanceId/instanceName query filter, which
is the legitimate source there. The per-instance-token path stays scoped
via findMany({ token, id }), so cross-instance access remains blocked.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@burhan-wesolves
burhan-wesolves merged commit 7abab20 into develop Aug 1, 2026
1 check passed
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.

2 participants