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]: mattermost-desktop appears not to respect DNS TTL #78

Closed
barryprice opened this issue Sep 27, 2023 · 2 comments
Closed

[Bug]: mattermost-desktop appears not to respect DNS TTL #78

barryprice opened this issue Sep 27, 2023 · 2 comments
Labels

Comments

@barryprice
Copy link

barryprice commented Sep 27, 2023

What happened?

We migrated a production Mattermost instance between data centres earlier today.

During the downtime period we intentionally took both source and target instances offline (so users would receive a 503 error) to avoid skew between the two installations during the sync.

Prior to and during this period, DNS TTL was reduce to 60s.

Once migration was complete, we restored service on the target instance but intentionally kept the source instance offline.

Connecting to the service via web browser worked fine at this point, but trying to use the already-running mattermost-desktop snap just showed 503 errors, confirmed by several users.

We tried logging out and logging in again, but this didn't make any difference.

It appears that the app does a DNS lookup on startup and then caches that result for far longer than expected, possibly indefinitely.

Fully stopping and relaunching the app resolved the problem.

What should have happened?

The app should have noticed that the target IP changed, and attempted to reconnect to the new target.

If not immediately, then certainly at the logout/login step.

Output of snap info $snap_name

name:      mattermost-desktop
summary:   Open source, private cloud Slack-alternative
publisher: Snapcrafters✪
store-url: https://snapcraft.io/mattermost-desktop
contact:   https://github.com/snapcrafters/mattermost-desktop/issues
license:   unset
description: |
  Mattermost is secure workplace messaging from behind your firewall.
  
    - Discuss topics in private groups, one-to-one or team-wide
    - Easily share and view image files
    - Connect in-house systems with webhooks and Slack-compatible integrations
  
  To use this app, you need a URL for a Mattermost server.
  
  -------
  
  Host your own server: https://about.mattermost.com/download
  Terms of Service: http://about.mattermost.com/terms/
  Contribute to the project: https://github.com/mattermost/desktop
  
  
  This snap is maintained by the Snapcrafters community, and is not necessarily endorsed or
  officially maintained by the upstream developers.
commands:
  - mattermost-desktop
snap-id:      ed0pxJoDHrgmAWHH7baX5nryAHy1UNj0
tracking:     latest/candidate
refresh-date: 6 days ago, at 21:22 +07
channels:
  latest/stable:    5.4.0 2023-06-22 (644) 111MB -
  latest/candidate: 5.5.0 2023-09-21 (687) 114MB -
  latest/beta:      ↑                            
  latest/edge:      5.5.0 2023-09-22 (689) 114MB -
installed:          5.5.0            (687) 114MB -

Output of snap connections $snap_name

Interface               Plug                                       Slot                            Notes
audio-playback          mattermost-desktop:audio-playback          :audio-playback                 -
audio-record            mattermost-desktop:audio-record            -                               -
camera                  mattermost-desktop:camera                  -                               -
content[gnome-42-2204]  mattermost-desktop:gnome-42-2204           gnome-42-2204:gnome-42-2204     -
content[gtk-3-themes]   mattermost-desktop:gtk-3-themes            gtk-common-themes:gtk-3-themes  -
content[icon-themes]    mattermost-desktop:icon-themes             gtk-common-themes:icon-themes   -
content[sound-themes]   mattermost-desktop:sound-themes            gtk-common-themes:sound-themes  -
desktop                 mattermost-desktop:desktop                 :desktop                        -
desktop-legacy          mattermost-desktop:desktop-legacy          :desktop-legacy                 -
gsettings               mattermost-desktop:gsettings               :gsettings                      -
home                    mattermost-desktop:home                    :home                           -
login-session-observe   mattermost-desktop:login-session-observe   -                               -
network                 mattermost-desktop:network                 :network                        -
network-bind            mattermost-desktop:network-bind            :network-bind                   -
opengl                  mattermost-desktop:opengl                  :opengl                         -
pulseaudio              mattermost-desktop:pulseaudio              :pulseaudio                     -
removable-media         mattermost-desktop:removable-media         -                               -
screen-inhibit-control  mattermost-desktop:screen-inhibit-control  :screen-inhibit-control         -
shared-memory           mattermost-desktop:shmem                   :shared-memory                  -
unity7                  mattermost-desktop:unity7                  :unity7                         -
wayland                 mattermost-desktop:wayland                 :wayland                        -
x11                     mattermost-desktop:x11                     :x11                            -

Output of snap version

snap    2.60.3+23.10.1
snapd   2.60.3+23.10.1
series  16
ubuntu  23.10
kernel  6.5.0-5-generic

Relevant log output

No response

Teminal output of app

No response

@barryprice barryprice added the bug label Sep 27, 2023
@merlijn-sebrechts
Copy link
Member

merlijn-sebrechts commented Sep 27, 2023 via email

@barryprice
Copy link
Author

Thanks, apologies for the noise - filed upstream here:

mattermost/desktop#2859

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

No branches or pull requests

2 participants