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]: Links to documentation in "Security & setup warnings" are outdated #42317

Closed
5 of 8 tasks
amlamarra opened this issue Dec 15, 2023 · 8 comments
Closed
5 of 8 tasks
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 28-feedback bug

Comments

@amlamarra
Copy link

⚠️ This issue respects the following points: ⚠️

Bug description

This has been a (minor) issue for a while and I figured I'd report it. I just updated to v28.0.0. The links to the documentation in "Security & setup warnings" (under Administration Settings) are outdated. For instance, this warning:

  • This instance is missing some recommended PHP modules. For improved performance and better compatibility it is highly recommended to install them: bz2. For more details see the documentation ↗.

Not only is it trying to link to version 9, but also "nextcloud.org" which hasn't worked in a while.

Steps to reproduce

  1. Click on your profile button in the top-right corner
  2. Go to "Administration settings"
  3. If there are any security & setup warnings, check the link to the documentation.

Expected behavior

Using the example I gave in the description, instead of linking to https://docs.nextcloud.org/server/9/go.php?to=admin-php-modules, it should probably link to https://docs.nextcloud.com/server/latest/go.php?to=admin-php-modules.

Installation method

Community Manual installation with Archive

Nextcloud Server version

28

Operating system

Debian/Ubuntu

PHP engine version

PHP 8.1

Web server

Apache (supported)

Database engine version

MariaDB

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

No response

List of activated Apps

No response

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

@amlamarra amlamarra added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Dec 15, 2023
@amlamarra
Copy link
Author

Forgot to mention, when I was searching to see if this was already reported, I only found 1 similar (if not the same) issue:
#15561

@joshtrichards
Copy link
Member

Hmm. Works for me. :-)

What does the OC_Version line contain in the /version.php located in your installation directory?

@amlamarra
Copy link
Author

$ grep OC_Version /var/www/nextcloud/version.php
$OC_Version = array(28,0,0,11);
$OC_VersionString = '28.0.0';
$OC_VersionCanBeUpgradedFrom = array (

@joshtrichards
Copy link
Member

joshtrichards commented Dec 17, 2023

Do you by chance having a custom theme active?

Because we don't have any references to docs.nextcloud.org anywhere in the code and the only other place I can see you could pick this up from is via ~/themes in you installation directory if you have your own from a long time ago. Possibly under occ config:list theming too.

@amlamarra
Copy link
Author

I was using the Breeze Dark theme before Nextcloud had a built-in dark theme. I just checked and it was still installed (though disabled since the update). I just uninstalled it and checked, but the links are still old. Any idea how I can fix this?

@joshtrichards
Copy link
Member

joshtrichards commented Dec 17, 2023

The only two places I can think of are stale parameters that might appear under:

occ theming:config
or
occ config:list theming

Also, check /var/www/html/themes (or equivalent, based on your installation folder) for anything other than a README and example folder.

@amlamarra
Copy link
Author

Those 2 commands did not show any stale values. However, I did see a defaults.php file and a core/ directory in my /var/www/nextcloud/themes directory. I removed those, refreshed the page and now the links are working correctly.

That defaults.php file has a class OC_Theme with a bunch of functions in there. These 2, in particular, are probably the culprits:

    /**
     * Returns the documentation URL
     * @return string URL
     */
    public function getDocBaseUrl() {
        return 'https://docs.nextcloud.org';
    }

    public function buildDocLinkToKey($key) {
        return $this->getDocBaseUrl() . '/server/9/go.php?to=' . $key;
    }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 28-feedback bug
Projects
None yet
Development

No branches or pull requests

4 participants
@joshtrichards @solracsf @amlamarra and others