Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Issue with upload data from healthleads and LAAC providers - data didn't update #881

Closed
abaranowska1 opened this issue Mar 18, 2020 · 1 comment
Assignees
Labels
bug Something isn't working Story Points: 3

Comments

@abaranowska1
Copy link

abaranowska1 commented Mar 18, 2020

During exploration tests, I found a bug related to uploading data. I had an issue with upload healthleads files - metadata.csv file is loading and loading.

metadata

The issue exists on both browsers Google Chrome Version 80.0.3987.132 and Firefox 74.0
What is more, even if I correctly upload all files, data didn't update for healthleads and LAAC providers. I tested upload also for the rest providers and didn't meet such issue.

Steps:

  1. Log in to the app as Admin
  2. Go to upload data
  3. Upload csv changed files for healthleads and LAAC providers
  4. Check if data update

Actual result: No update for healthleads and LAAC providers
Expected result: Data should be updated after upload files.

Dev notes:

2020-03-18 09:16:01.314 ERROR 1 --- [  XNIO-1 task-1] io.undertow.request                      : UT005023: Exception handling request to /nice%20ports%2C/Tri%6Eity.txt%2ebak
org.springframework.security.web.firewall.RequestRejectedException: The request was rejected because the URL contained a potentially malicious String "%2e"
2020-03-18 10:46:09.071  INFO 1 --- [ XNIO-1 task-14] o.b.s.s.impl.DocumentUploadServiceImpl   : Data upload for LAAC has started
2020-03-18 10:46:09.491 ERROR 1 --- [ XNIO-1 task-14] o.b.s.manager.ImportManagerImpl          : org.hibernate.TransientObjectException: object references an unsaved transient instance - save the transient instance before flushing: org.benetech.servicenet.domain.GeocodingResult; nested exception is java.lang.IllegalStateException: org.hibernate.TransientObjectException: object references an unsaved transient instance - save the transient instance before flushing: org.benetech.servicenet.domain.GeocodingResult
at com.sun.proxy.$Proxy304.findOneByExternalDbIdAndProviderName(Unknown Source)
at org.benetech.servicenet.service.impl.ServiceTaxonomyServiceImpl.findForExternalDb(ServiceTaxonomyServiceImpl.java:99)
at org.benetech.servicenet.service.impl.ServiceTaxonomyServiceImpl$$FastClassBySpringCGLIB$$3eaedd92.invoke(<generated>)
at org.benetech.servicenet.service.impl.ServiceBasedImportServiceImpl$$EnhancerBySpringCGLIB$$c32e15d9.createOrUpdateServiceTaxonomy(<generated>)
@abaranowska1 abaranowska1 created this issue from a note in Soldevelo Project Board (Tech Debt Backlog) Mar 18, 2020
@abaranowska1 abaranowska1 added the bug Something isn't working label Mar 18, 2020
@johnhbenetech johnhbenetech moved this from Tech Debt Backlog to Up Next in Soldevelo Project Board Mar 31, 2020
@kdondziak kdondziak moved this from Up Next to In progress in Soldevelo Project Board Mar 31, 2020
@kdondziak kdondziak self-assigned this Mar 31, 2020
kdondziak added a commit that referenced this issue Apr 3, 2020
kdondziak added a commit that referenced this issue Apr 3, 2020
jkondrat added a commit that referenced this issue Apr 3, 2020
 #881: Fix upload issue related to saving geocoding results
kdondziak added a commit that referenced this issue Apr 6, 2020
@kdondziak kdondziak moved this from In progress to Waiting for review in Soldevelo Project Board Apr 6, 2020
kdondziak added a commit that referenced this issue Apr 6, 2020
@kdondziak kdondziak moved this from Waiting for review to QA in Soldevelo Project Board Apr 6, 2020
@abaranowska1
Copy link
Author

abaranowska1 commented Apr 6, 2020

Tested with success

Verified on:

  • Google Chrome Version 80.0.3987.149

Verified cases:

  • Upload csv changed files for healthleads and LAAC providers
  • Upload changed files for all providers
  • Check if data update

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working Story Points: 3
Projects
Development

No branches or pull requests

2 participants