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

/thirdparty/protocols API returned an empty object for an appservice that wasn't ready #9694

Closed
jaller94 opened this issue Mar 26, 2021 · 3 comments · Fixed by #10532
Closed
Labels
S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.

Comments

@jaller94
Copy link
Contributor

jaller94 commented Mar 26, 2021

Description

/thirdparty/protocols returned

{ irc: {} }

The irc object is missing a few required values.

https://matrix.org/docs/spec/client_server/r0.6.1#get-matrix-client-r0-thirdparty-protocols

Steps to reproduce

  • I set up my IRC bridge but the pingback port wasn't working yet (I had set it to the wrong port in the appservice file) and the IRC server on localhost wasn't connecting yet.

Synapse-config, Bridge config and logs (look around the time 2021-03-25 20:44):
https://gateway.pinata.cloud/ipfs/QmTbjP9PwfaTELkYQQ1HmznR4n8GNusveE6BwpZXw5tjk4

Version information

Additional notes:

The API response caused an issue in Element Web because it's not spec-compliant.

element-hq/element-web#16789

Peek.2021-03-25.20-45.mp4
@jaller94 jaller94 changed the title /thirdparty/protocols API returned an empty object /thirdparty/protocols API returned an empty object for an appservice that wasn't ready Mar 26, 2021
@jaller94 jaller94 added the T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. label Mar 26, 2021
@clokep clokep added the S-Minor Blocks non-critical functionality, workarounds exist. label Mar 29, 2021
@cremesk
Copy link
Contributor

cremesk commented Jul 29, 2021

@jaller94 You can delete the protocols section of the registration file, if you are running your own irc bridge.
after this you need to restart synapse.

This fixed the Bug Temporary and disabled the "IRC-Network"-Menu.

eg.: "TildeChat"
2021-07-07_21-04

brief explanation of how i came up with this:

i got is problem after i enables the "dynamic channels" feature in appservice-irc-bridge.
the bridge add an area to the local room explorer which contains the irc-rooms.

the room-explorer is not longer working locally but it works from other instances..

my server is returning an invalid API response to /protocols - instances is marked as required but my is returning undefined.
see: https://matrix.envs.net/_matrix/media/r0/download/envs.net/3d67c2324bf239a8e38ba3b02ebfe04ee908ff79

@Half-Shot
Copy link
Collaborator

The bug here is that synapse shouldn't return results for bridges it can't reach on this endpoint, rather than a bug in the bridges themselves.

@skepticalwaves
Copy link

Looks like https://github.com/Half-Shot/matrix-appservice-discord currently returns an empty protocol as well

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants