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

Proposal Add /_matrix/media/v1/resolve_url to Client-Server API: download and preview urls in the clients despite CORS #1270

Closed
oivoodoo opened this issue May 31, 2018 · 6 comments

Comments

@oivoodoo
Copy link

@oivoodoo oivoodoo commented May 31, 2018

@turt2live

This comment has been minimized.

Copy link
Member

@turt2live turt2live commented Jun 2, 2018

Related: #608

@uhoreg uhoreg changed the title Synapse /_matrix/media/v1/resolve_url to API download and preview urls in the clients despite on CORS Add /_matrix/media/v1/resolve_url to Client-Server API: download and preview urls in the clients despite CORS Jun 4, 2018
@uhoreg

This comment has been minimized.

Copy link
Member

@uhoreg uhoreg commented Jun 4, 2018

(clarified title a bit -- in particular, this is not Synapse-specific, but would apply to any Matrix Homeserver.)

@oivoodoo

This comment has been minimized.

Copy link
Author

@oivoodoo oivoodoo commented Jun 5, 2018

@uhoreg thank you, now it looks better. what do you think about the new endpoint? this concept has already implemented in my branches and ready to give a try in case if spec looks good for everyone.

@oivoodoo

This comment has been minimized.

Copy link
Author

@oivoodoo oivoodoo commented Jun 11, 2018

Hi @richvdh. Could you review the spec file as well? thank you!

@oivoodoo oivoodoo changed the title Add /_matrix/media/v1/resolve_url to Client-Server API: download and preview urls in the clients despite CORS Proposal Add /_matrix/media/v1/resolve_url to Client-Server API: download and preview urls in the clients despite CORS Jun 19, 2018
@turt2live turt2live added the T-Core label Dec 24, 2018
@anoadragon453 anoadragon453 removed the T-Core label Jan 4, 2019
@uhoreg

This comment has been minimized.

Copy link
Member

@uhoreg uhoreg commented Nov 29, 2019

I've re-written the Google doc as a markdown doc at #2370

@turt2live

This comment has been minimized.

Copy link
Member

@turt2live turt2live commented Nov 29, 2019

Flagging this one as obsolete to continue over at that issue then,

@turt2live turt2live closed this Nov 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.