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

TRPEndpoint not shown in the interface, even though it is set. #246

Open
ping-localhost opened this issue Nov 22, 2024 · 5 comments
Open

Comments

@ping-localhost
Copy link
Contributor

At /about the TRP Endpoint is shown as N/A, even though it is set in the env. In fact, TRPEndpoint seems to not be used anywhere in the code.

image

@daniellemaxwell
Copy link
Collaborator

@ping-localhost The N/A value for TRP Endpoint on the About page is currently a hardcoded value. We will remove the need to have separate TRISA AND TRP endpoints soon. Once the changes are merged, we'll follow up and let you know.

@ping-localhost
Copy link
Contributor Author

ping-localhost commented Jan 9, 2025

@daniellemaxwell Has this change been merged already and released? We are seeing the following:

{"level":"warn","path":"/","ser_name":"trp","version":"0.27.1-beta.33 (a8a5eb7)","method":"GET","resp_time":0.161824,"resp_bytes":83,"status":404,"client_ip":"87.236.176.241","request_id":"01JH50VGHEM30E9VQDAAWZ67ND","severity":"WARNING","time":"2025-01-09T07:53:23Z","message":"trp GET / 404"}

A 404 does not seem like something we want. The node switched to TRISA, but we do need to be able to get TRP messages.

@bbengfort
Copy link
Collaborator

@daniellemaxwell Has this change been merged already and released? We are seeing the following:

{"level":"warn","path":"/","ser_name":"trp","version":"0.27.1-beta.33 (a8a5eb7)","method":"GET","resp_time":0.161824,"resp_bytes":83,"status":404,"client_ip":"87.236.176.241","request_id":"01JH50VGHEM30E9VQDAAWZ67ND","severity":"WARNING","time":"2025-01-09T07:53:23Z","message":"trp GET / 404"}

A 404 does not seem like something we want. The node switched to TRISA, but we do need to be able to get TRP messages.

@ping-localhost unfortunately the fix is not merged in quite yet; we need to move to a system that has only one endpoint for everything rather than 3 in order to make the travel addresses work; but that came with a few unseen complications. Are you using the webhook? I can wire something up that would have incoming messages get passed to the webhook; but they might not appear in the UI or in the API; would that work as a temporary solution for you to start receiving messages right away?

@ping-localhost
Copy link
Contributor Author

We are currently using the API to fetch all the data we need (so it missing from the API would be a deal breaker right now). The webhook is not being used.

@bbengfort
Copy link
Collaborator

Alright - let me see if I can get something together for you next week - I don't want you to be blocked by the one port thing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants