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

[Bug] Visit link missing port #385

Open
reedy opened this issue Nov 27, 2024 · 8 comments
Open

[Bug] Visit link missing port #385

reedy opened this issue Nov 27, 2024 · 8 comments
Labels
bug Something isn't working

Comments

@reedy
Copy link

reedy commented Nov 27, 2024

Describe the issue

The router I have setup is set on a public IPv4 address, on a port that isn't port 80.

On config/devices/device/, the visit link seems to not use that port

How to reproduce the issue

Create a config that includes a non default port (ie not port 80), and try and use the visit link to get to the web UI of the 'tik device

Expected behavior

The visit link to include the correct port...

Screenshots

Screenshot 2024-11-27 at 21 08 48

Software versions

  • Home Assistant version: 2024.11.3
  • Mikrotik Router integration version: 2.1.4
  • Mikrotik Hardware: SXT R (not specific)
  • RouterOS version: 7.16.1

Diagnostics data

Traceback/Error logs

Additional context

@reedy reedy added the bug Something isn't working label Nov 27, 2024
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale label Dec 12, 2024
@reedy
Copy link
Author

reedy commented Dec 12, 2024

Still an issue..

@github-actions github-actions bot removed the stale label Dec 13, 2024
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale label Dec 28, 2024
@reedy
Copy link
Author

reedy commented Dec 30, 2024

This stale bot is far too eager.

@github-actions github-actions bot removed the stale label Dec 31, 2024
@tomaae
Copy link
Owner

tomaae commented Jan 8, 2025

I agree, this new stale bot is annoying. But old well working one want obsolete in favor of this one.
Can you submit your diagnostic data? I'm mostly interested in setup section. Data from device are not necessary.

@reedy
Copy link
Author

reedy commented Jan 8, 2025

Can you submit your diagnostic data? I'm mostly interested in setup section. Data from device are not necessary.

Just the json from "Download Diagnostics"? Or something after following https://github.com/tomaae/homeassistant-mikrotik_router?tab=readme-ov-file#enabling-debug ?

Copy link
Owner

tomaae commented Jan 8, 2025

download diagnostics
just "custom_components>data>entry" is enough:

  "data": {
    "entry": {
      "data": {
        "host": "192.168.0.1",
        "name": "myrouter",
        "password": "**REDACTED**",
        "port": 0,
        "ssl": true,
        "username": "**REDACTED**"
      },

@reedy
Copy link
Author

reedy commented Jan 8, 2025

  "data": {
    "entry": {
      "data": {
        "host": "81.xxx.xxx.xxx",
        "name": "Mikrotik",
        "password": "**REDACTED**",
        "port": 8728,
        "ssl": false,
        "username": "**REDACTED**"
      },
      "options": {}
    },

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants