Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Remove support for legacy application service paths #15380

Closed
clokep opened this issue Apr 3, 2023 · 1 comment · Fixed by #15964
Closed

Remove support for legacy application service paths #15380

clokep opened this issue Apr 3, 2023 · 1 comment · Fixed by #15964
Labels
A-Application-Service Related to AS support O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Blocks non-critical functionality, workarounds exist. T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. Z-Future-Maintenance Things that can't yet be done, but will need cleaning up in a couple of months/releases
Milestone
Revisit: Next Week

Comments

@clokep
Copy link
Member

clokep commented Apr 3, 2023

#15317 adds support for using the /_matrix/app/v1 paths instead of the unprefixed (or unstable) paths for application services.

It declares the old pre-specced versions as deprecated and sets a removal date at or after Synapse v1.88.0.
@clokep clokep added S-Minor Blocks non-critical functionality, workarounds exist. T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. Z-Future-Maintenance Things that can't yet be done, but will need cleaning up in a couple of months/releases O-Uncommon Most users are unlikely to come across this or unexpected workflow labels Apr 3, 2023
@clokep
Copy link
Member Author

clokep commented Apr 3, 2023

(Note that I don't think this has any spec issues like #15379 because the "legacy" endpoints are from before any versioned spec.)

@MadLittleMods MadLittleMods added the A-Application-Service Related to AS support label May 25, 2023
@clokep clokep changed the title Remove support for legacy application service paths Remove support for legacy application service paths, access tokens, and user registration Jun 1, 2023
@clokep clokep changed the title Remove support for legacy application service paths, access tokens, and user registration Remove support for legacy application service paths Jun 1, 2023
@H-Shay H-Shay mentioned this issue Jul 19, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-Application-Service Related to AS support O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Blocks non-critical functionality, workarounds exist. T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. Z-Future-Maintenance Things that can't yet be done, but will need cleaning up in a couple of months/releases
Projects
None yet
Milestone
Revisit: Next Week
Development

Successfully merging a pull request may close this issue.

Remove support for legacy application service paths matrix-org/synapse
3 participants
@clokep @MadLittleMods @erikjohnston