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

Add Taipy-specific and hidden parameters to navigate() #2388

Open
1 of 7 tasks
FabienLelaquais opened this issue Jan 9, 2025 · 0 comments
Open
1 of 7 tasks

Add Taipy-specific and hidden parameters to navigate() #2388

FabienLelaquais opened this issue Jan 9, 2025 · 0 comments
Labels
🖰 GUI Related to GUI ✨New feature 🟧 Priority: High Must be addressed as soon 📝Release Notes Impacts the Release Notes or the Documentation in general

Comments

@FabienLelaquais
Copy link
Member

FabienLelaquais commented Jan 9, 2025

Description

We had a request to allow a Taipy application to query another Taipy application page, potentially served by another Taipy GUI application.
This can be done using the params parameter to navigate() but then the URL is complemented with the query string that the requestee is willing to hide, for security reasons.

Solution Proposed

navigate() will be added a hidden_params parameter, with the same semantics as for params: a dictionary that holds string keys and serializable value.
The on_navigate callback will receive an additional hidden_params parameter, set to a dictionary that reflects what was provided in the call to navigate().

Impact of Solution

Those parameters are encoded in the GET query header, so no POST is mandatory.
In order not to unintendedly overwrite the original headers configuration, each hidden parameter is propagated with an internal (and undocumented) prefix that identifies them before they're encoded in the query header.
Taipy GUI will decypher all that before on_navigate is triggered.

Acceptance Criteria

  • If applicable, a new demo code is provided to show the new feature in action.
  • Integration tests exhibiting how the functionality works are added.
  • Any new code is covered by a unit tested.
  • Check code coverage is at least 90%.
  • Related issue(s) in taipy-doc are created for documentation and Release Notes are updated.

Code of Conduct

  • I have checked the existing issues.
  • I am willing to work on this issue (optional)
@FabienLelaquais FabienLelaquais added 🖰 GUI Related to GUI 🟧 Priority: High Must be addressed as soon ✨New feature 📝Release Notes Impacts the Release Notes or the Documentation in general labels Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖰 GUI Related to GUI ✨New feature 🟧 Priority: High Must be addressed as soon 📝Release Notes Impacts the Release Notes or the Documentation in general
Projects
None yet
Development

No branches or pull requests

1 participant