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

Exception thrown disabling external sites app #16297

Closed
jnfrmarks opened this issue May 12, 2015 · 8 comments · Fixed by owncloud-archive/apps#2063
Closed

Exception thrown disabling external sites app #16297

jnfrmarks opened this issue May 12, 2015 · 8 comments · Fixed by owncloud-archive/apps#2063
Assignees
Milestone

Comments

@jnfrmarks
Copy link

Steps to reproduce

  1. Install 8.1
  2. Enable External sites app & configure
  3. Disable External Sites app

Expected behaviour

No errors in the log file

Actual behaviour

I'm seeing this error in the log file:


{"reqId":"LP9wygxfxtsL4PBEirCj","remoteAddr":"x.x.x.x","app":"index","message":"Exception: {\"Exception\":\"Symfony\\\\Component\\\\Routing\\\\Exception\\\\RouteNotFoundException\",\"Message\":\"Unable to generate a URL for the named route \\\"external_index\\\" as such route does not exist.\",\"Code\":0,\"Trace\":\"#0 \\\/var\\\/www\\\/html\\\/owncloud\\\/lib\\\/private\\\/route\\\/router.php(304): Symfony\\\\Component\\\\Routing\\\\Generator\\\\UrlGenerator->generate('external_index', Array, false)\\n#1 \\\/var\\\/www\\\/html\\\/owncloud\\\/lib\\\/private\\\/urlgenerator.php(65): OC\\\\Route\\\\Router->generate('external_index', Array)\\n#2 \\\/var\\\/www\\\/html\\\/owncloud\\\/lib\\\/private\\\/helper.php(139): OC\\\\URLGenerator->linkToRoute('external_index', Array)\\n#3 \\\/var\\\/www\\\/html\\\/owncloud\\\/lib\\\/public\\\/util.php(286): OC_Helper::linkToRoute('external_index', Array)\\n#4 \\\/var\\\/www\\\/html\\\/owncloud\\\/apps\\\/external\\\/appinfo\\\/app.php(34): OCP\\\\Util::linkToRoute('external_index', Array)\\n#5 \\\/var\\\/www\\\/html\\\/owncloud\\\/lib\\\/private\\\/app.php(389): require('\\\/var\\\/www\\\/html\\\/o...')\\n#6 \\\/var\\\/www\\\/html\\\/owncloud\\\/settings\\\/ajax\\\/navigationdetect.php(30): OC_App::getAppNavigationEntries('external')\\n#7 \\\/var\\\/www\\\/html\\\/owncloud\\\/lib\\\/private\\\/route\\\/route.php(153) : runtime-created function(1): require_once('\\\/var\\\/www\\\/html\\\/o...')\\n#8 [internal function]: __lambda_func(Array)\\n#9 \\\/var\\\/www\\\/html\\\/owncloud\\\/lib\\\/private\\\/route\\\/router.php(272): call_user_func('?lambda_7358', Array)\\n#10 \\\/var\\\/www\\\/html\\\/owncloud\\\/lib\\\/base.php(860): OC\\\\Route\\\\Router->match('\\\/settings\\\/ajax\\\/...')\\n#11 \\\/var\\\/www\\\/html\\\/owncloud\\\/index.php(40): OC::handleRequest()\\n#12 {main}\",\"File\":\"\\\/var\\\/www\\\/html\\\/owncloud\\\/3rdparty\\\/symfony\\\/routing\\\/Symfony\\\/Component\\\/Routing\\\/Generator\\\/UrlGenerator.php\",\"Line\":134}","level":4,"time":"2015-05-12T19:02:48+00:00"}

Server Host info:

Ubuntu 14.04
PHP 5.5.9

Server configuration:

ownCloud Enterprise Edition 8.1 beta 1 (daily) Build:2015-05-12T03:08:46+00:00 20d2d8d

@jnfrmarks jnfrmarks added this to the 8.1-current milestone May 12, 2015
@karlitschek
Copy link
Contributor

strange @DeepDiver1975 what do you think?

@DeepDiver1975
Copy link
Member

We have seen similar log messages for activity.
There is no really much we can do. All this exception is telling us: somebody requested an url which we don't know (anymore).

This can happen anytime - in this case there is/was still someone trying to access the url of the just disabled app.

@nickvergessen
Copy link
Contributor

In this case there is something we can do. Will create a PR on the externals app

@nickvergessen
Copy link
Contributor

Fixed with owncloud-archive/apps#2063

@MorrisJobke
Copy link
Contributor

@DeepDiver1975 Doesn't this sound familiar? I reported this over a year ago ;) #8361

@MorrisJobke
Copy link
Contributor

The fixed behaviour just is a side effect of the actual code change, because it is now executed only if the nav entry is needed ;) I will try this with music and news app again ;)

@MorrisJobke
Copy link
Contributor

The actual problem here is, that we somehow load apps even if they are disabled during the very same request.

@MorrisJobke
Copy link
Contributor

This doesn't seem to occur anymore in master.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants