opensondage_ynh icon indicating copy to clipboard operation
opensondage_ynh copied to clipboard

Domain 'mydomain.bug' unknown

Open Jioleoo opened this issue 4 years ago • 2 comments

Describe the bug

The install bug due to the wrong domain name used. My domain is somethine like 'my-domain.test' and the one that's not found is 'my.domain.test'

For me it's clear that it's due to this line in the install script :

# Modify the domain to be used in a regex
domain_regex=$(echo "$domain" | sed 's@-@.@g')

Context

  • Hardware: VPS
  • YunoHost version: 4.1.7.1
  • I have access to my server: Through SSH | through the webadmin
  • Are you in a special context or did you perform some particular tweaking on your YunoHost instance?: no
  • Using, or trying to install package version/branch: lastest

Steps to reproduce

  • use a domain with "-" in it.
  • try to install OpenSondage.

Jioleoo avatar Feb 09 '21 11:02 Jioleoo

Hi, same problem here, with an existing Opensondage instance. The upgrade fails because of 'Domain foo.domain.tld unknown'. The domain currently used by Opensondage is 'foo-domain.tld'.

SiM avatar May 18 '21 15:05 SiM

Same here. And indeed after a small investigating, there is a small bug around https://github.com/YunoHost-Apps/opensondage_ynh/blob/c28adad8ae4914bea47765d23e4c9492748c7400/scripts/install#L175.

The workaround for now is :

  • install Open sondage without public mode
  • after the installation succeed, add Open Sondage to the All Users persmission
  • 🎉

But next upgrade, it could fail because the buggy line also exists for upgrade case https://github.com/YunoHost-Apps/opensondage_ynh/blob/c28adad8ae4914bea47765d23e4c9492748c7400/scripts/upgrade#L231.

lenybernard avatar May 18 '21 19:05 lenybernard