Project

General

Profile

Actions
Copy link

bug #8804

closed

taxeditor hangs when creating a new classification

Added by Andreas Kohlbecker over 4 years ago. Updated over 4 years ago.

Status:
Worksforme
Priority:
New
Assignee:
Katja Luther
Category:
taxeditor
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Severity:
critical
Found in Version:

Description

[caryophyllales_spp] 13:04:59,556 DEBUG [qtp759156157-135][or.sp.we.se.DispatcherServlet] - DispatcherServlet with name 'cdmlib-remote' processing POST request for [/remoting/classification.service]
[caryophyllales_spp] 13:04:59,556 DEBUG [qtp759156157-135][or.sp.we.se.ha.SimpleUrlHandlerMapping] - Mapping [/remoting/classification.service] to HandlerExecutionChain with handler [org.springframework.remoting.httpinvoker.HttpInvokerServiceExporter@212acf44] and 1 interceptor
[caryophyllales_spp] 13:04:59,558 DEBUG [qtp759156157-135][eu.et.cd.pe.hi.pe.CdmPermissionEvaluator] - starting evaluation => ...
[caryophyllales_spp] 13:04:59,558 DEBUG [qtp759156157-135][eu.et.cd.pe.hi.pe.CdmPermissionEvaluator] - ROLE_ADMIN found => true
[caryophyllales_spp] 13:04:59,559 DEBUG [qtp759156157-135][eu.et.cd.pe.hi.pe.CdmPermissionEvaluator] - starting evaluation => ...
[caryophyllales_spp] 13:04:59,559 DEBUG [qtp759156157-135][eu.et.cd.pe.hi.pe.CdmPermissionEvaluator] - ROLE_ADMIN found => true
[caryophyllales_spp] 13:04:59,559 DEBUG [qtp759156157-135][eu.et.cd.pe.hi.pe.CdmPermissionEvaluator] - starting evaluation => ...
[caryophyllales_spp] 13:04:59,559 DEBUG [qtp759156157-135][eu.et.cd.pe.hi.pe.CdmPermissionEvaluator] - ROLE_ADMIN found => true
[caryophyllales_spp] 13:04:59,559 DEBUG [qtp759156157-135][eu.et.cd.pe.hi.pe.CdmPermissionEvaluator] - starting evaluation => ...
[caryophyllales_spp] 13:04:59,559 DEBUG [qtp759156157-135][eu.et.cd.pe.hi.pe.CdmPermissionEvaluator] - ROLE_ADMIN found => true

The same problems also blocks

  1. adding a new taxon to an empty classification
  2. removing the last taxon from a classification
Actions
Copy link
 #1

Updated by Andreas Kohlbecker over 4 years ago

I put this into the current milestone since it is really an awful regression.
This issue is really critical and maybe easy to fix.

Actions
Copy link
 #2

Updated by Andreas Kohlbecker over 4 years ago

  • Description updated (diff)
Actions
Copy link
 #3

Updated by Andreas Kohlbecker over 4 years ago

  • Description updated (diff)
Actions
Copy link
 #4

Updated by Andreas Müller over 4 years ago

I can't reproduce this. Can you give more information. I created classifications in multiple DBs and it worked.
Which DB did you use on which sever with which rights?

Actions
Copy link
 #5

Updated by Andreas Kohlbecker over 4 years ago

Andreas Müller wrote:

I can't reproduce this. Can you give more information. I created classifications in multiple DBs and it worked.
Which DB did you use on which sever with which rights?

I tested on a local copy of the edit-test/caryophyllales_spp from which i took a dump in Dez. 2019

I connected to taxeditor to a cdmlib-remote-webapp instance at the revision in the develop branch:

* cc5e8c6ee9 - Tue, 14 Jan 2020 15:56:57 +0100 (22 hours ago) 
|   fixing bugs in SwaggerIT and StaticSwaggerApiDoc - Andreas Kohlbecker

I was logged in a admin

Actions
Copy link
 #6

Updated by Andreas Müller over 4 years ago

Can you please test on test/nightly also. This is our reference system for ticket reviews etc. Ofcourse we also need to find out why it does not work on your computer with the dump but most important (also for the question if this is a "critical" ticket and needs to be fixed in current milestone) is if it can be reproduced on the server.

Actions
Copy link
 #7

Updated by Andreas Kohlbecker over 4 years ago

I tested again locally with another DB where it also occurs ....
will test also on the test server, but this will require to build the cdm-server and to install it ... so this will take a while ...

why don't you test on nightly on test server? Or do you think it is a regression from today?

Actions
Copy link
 #8

Updated by Katja Luther over 4 years ago

I can't reproduce it on my local db. And the recursion seems to be something on library side in PermissionEvaluator

Actions
Copy link
 #9

Updated by Andreas Kohlbecker over 4 years ago

  • Status changed from New to Worksforme
  • Target version deleted (Release 5.12)

Reproducible with the latest cdm-server and Taxeditor Version: 5.12.0-SNAPSHOT, Build Date: 2020-01-15 11:50 CET on http://test.e-taxonomy.eu/cdmserver/caryo_spp/

No longer reproducible with Taxeditor Version: 5.12.0-SNAPSHOT, Build Date: 2020-01-15 13:48 CET

Reproducible with Taxeditor Version: 5.12.0-SNAPSHOT, Build Date: 2020-01-15 13:48 CET at cdmlib cc5e8c6ee9:

No longer reproducible with Taxeditor Version: 5.12.0-SNAPSHOT, Build Date: 2020-01-15 13:48 CET at cdmlib 8841d740c6

So the bug has been fixed by one of the below commits:

* 8841d740c6 - Wed, 15 Jan 2020 13:49:38 +0100 (27 minutes ago) (origin/develop)
|   adapted tests for harmonization of taxonnodeDto usage - Katja Luther
* b140cc5433 - Wed, 15 Jan 2020 13:48:54 +0100 (27 minutes ago)
|   merge ITaxonNodeService - Katja Luther
* 99f733939c - Wed, 15 Jan 2020 13:42:15 +0100 (34 minutes ago)
|   cleanup taxonnodeDto - UuidAndTitleCache<TaxonNode> - Katja Luther
* 6ad1062d3b - Wed, 15 Jan 2020 11:12:00 +0100 (3 hours ago)
|   cleanup - Andreas Müller
* a65a935cd5 - Wed, 15 Jan 2020 11:08:55 +0100 (3 hours ago)
|   fix #8803 add per mm² to MeasurementUnit - Andreas Müller
* 5675e6119c - Wed, 15 Jan 2020 11:05:15 +0100 (3 hours ago)
|   fix getPage() for deduplication - Andreas Müller
* 30df4b0ad8 - Wed, 15 Jan 2020 11:02:24 +0100 (3 hours ago)
|   cleanup - Andreas Müller
* cc5e8c6ee9 - Tue, 14 Jan 2020 15:56:57 +0100 (22 hours ago) (HEAD -> develop)
Actions
Copy link

Also available in: Atom PDF