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

webapp always sets db ping option #918

Open
white-haired-uncle opened this issue Jun 18, 2024 · 1 comment
Open

webapp always sets db ping option #918

white-haired-uncle opened this issue Jun 18, 2024 · 1 comment
Assignees

Comments

@white-haired-uncle
Copy link

  • Platform:
    Linux mythbackend 6.5.0-35-generic #35~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Tue May 7 09:00:52 UTC 2 x86_64 x86_64 x86_64 GNU/Linux

  • MythTV version:

MythTV Version : v34.0-18-g5590d0d5eb
MythTV Branch : fixes/34
Network Protocol : 91
Library API : 34.20220913-1
QT Version : 5.15.3
Options compiled in:
 linux profile use_hidesyms using_alsa using_oss using_pulse using_pulseoutput using_backend using_bindings_perl using_bindings_python using_bindings_php using_dvb using_firewire using_frontend using_hdhomerun using_satip using_vbox using_ceton using_joystick_menu using_libcec using_libcrypto using_gnutls using_libdns_sd using_libxml2 using_lirc using_mheg using_opengl using_egl using_qtwebkit using_qtscript using_qtdbus using_taglib using_v4l2 using_v4l2prime using_x11 using_system_libbluray using_system_libudfread using_systemd_notify using_systemd_journal using_drm using_bindings_perl using_bindings_python using_bindings_php using_freetype2 using_mythtranscode using_opengl using_egl using_drm using_vaapi using_nvdec using_vdpau using_ffmpeg_threads using_mheg using_libass using_libxml2 using_libmp3lame

  • Package version:

  • Component:
    webapp

What steps will reproduce the bug?

  1. Start with the option to ping the db host disabled in ~mythtv/.mythtv/config.xml.
  2. Make any valid change in http://mythbackend:6544/setupwizard/dbsetup and save
  3. Note that <PingHost> is now set to true.

How often does it reproduce? Is there a required condition?

Always for me.

What is the expected behaviour?

The option to enable/disable ping of the db host should be available in webapp, and saved correctly in config.xml.

What do you see instead?

Additional information

While not common in the kind of environments mythtv is going to be installed in, a host not responding to ping is (or was last time I looked) a perfectly valid condition.

@bennettpeter bennettpeter self-assigned this Jun 18, 2024
@bennettpeter
Copy link
Member

The ping option is no longer used. Since 2017 the code uses a port check rather than a ping, and always does the check, regardless of the setting. The ping check should really be removed from the mythfrontend setup and from the config.xml file. I don't know why the web app is always setting it true.

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

No branches or pull requests

2 participants