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

Enhancement/improvement: display alert if web search fetcher cannot succeed, instead of exception #6343

Closed
ilippert opened this issue Apr 25, 2020 · 7 comments

Comments

@ilippert
Copy link
Contributor

ilippert commented Apr 25, 2020

reframing this as a small improvement relating to #6331

Problem
I am always frustrated when I use google web search and get an exception - because I never immediately know what the issue is.

Proposed solution
I would appreciate a specific error message (rather than getting an exception) "[X] is preventing you from more searches." or something similar. [X] is the Fetcher, e.g. google or ieee.

Log File org.jabref.logic.importer.FetcherException: Error while fetching from Google Scholar at org.jabref/org.jabref.logic.importer.fetcher.GoogleScholar.performSearch(Unknown Source) at org.jabref/org.jabref.gui.importer.fetcher.WebSearchPaneViewModel.lambda$search$1(Unknown Source) at org.jabref/org.jabref.gui.util.BackgroundTask$1.call(Unknown Source) at org.jabref/org.jabref.gui.util.DefaultTaskExecutor$1.call(Unknown Source) at org.jabref.merged.module/javafx.concurrent.Task$TaskCallable.call(Unknown Source) at java.base/java.util.concurrent.FutureTask.run(Unknown Source) at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source) at java.base/java.util.concurrent.FutureTask.run(Unknown Source) at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source) at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) at java.base/java.lang.Thread.run(Unknown Source) Caused by: java.io.IOException: Server returned HTTP response code: 403 for URL: https://scholar.googleusercontent.com/scholar.bib?q=info:fu-HqA2D0GoJ:scholar.google.com/&output=citation&scisdr=CgWDtGZeEK2YzlELkXM:AAGBfm0AAAAAXqQOiXNFaz5GoAYFVPqNRM7F4RqXPnnc&scisig=AAGBfm0AAAAAXqQOiduRwxTLbjmUUnOU60Ku7REjtsyZ&scisf=4&ct=citation&cd=17&hl=en at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(Unknown Source) at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(Unknown Source) at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Unknown Source) at java.base/java.lang.reflect.Constructor.newInstance(Unknown Source) at java.base/sun.net.www.protocol.http.HttpURLConnection$10.run(Unknown Source) at java.base/sun.net.www.protocol.http.HttpURLConnection$10.run(Unknown Source) at java.base/java.security.AccessController.doPrivileged(Unknown Source) at java.base/sun.net.www.protocol.http.HttpURLConnection.getChainedException(Unknown Source) at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream0(Unknown Source) at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream(Unknown Source) at java.base/sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(Unknown Source) at org.jabref/org.jabref.logic.net.URLDownload.asString(Unknown Source) at org.jabref/org.jabref.logic.net.URLDownload.asString(Unknown Source) at org.jabref/org.jabref.logic.importer.fetcher.GoogleScholar.downloadEntry(Unknown Source) at org.jabref/org.jabref.logic.importer.fetcher.GoogleScholar.addHitsFromQuery(Unknown Source) ... 11 more Caused by: java.io.IOException: Server returned HTTP response code: 403 for URL: https://scholar.googleusercontent.com/scholar.bib?q=info:fu-HqA2D0GoJ:scholar.google.com/&output=citation&scisdr=CgWDtGZeEK2YzlELkXM:AAGBfm0AAAAAXqQOiXNFaz5GoAYFVPqNRM7F4RqXPnnc&scisig=AAGBfm0AAAAAXqQOiduRwxTLbjmUUnOU60Ku7REjtsyZ&scisf=4&ct=citation&cd=17&hl=en at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream0(Unknown Source) at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream(Unknown Source) at java.base/java.net.HttpURLConnection.getResponseCode(Unknown Source) at java.base/sun.net.www.protocol.https.HttpsURLConnectionImpl.getResponseCode(Unknown Source) at org.jabref/org.jabref.logic.net.URLDownload.openConnection(Unknown Source) ... 15 more
@ilippert ilippert changed the title missing specific error message on known google scholar issue request for improvement: display error message on known google scholar issue May 1, 2020
@ilippert
Copy link
Contributor Author

ilippert commented May 2, 2020

I think, this suggestion comes close to the suggestion at #6376

@ilippert ilippert changed the title request for improvement: display error message on known google scholar issue Enhancement/improvement: display alert if web search fetcher cannot succeed, instead of exception May 2, 2020
@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2020

This issue has been inactive for half a year. Since JabRef is constantly evolving this issue may not be relevant any longer and it will be closed in two weeks if no further activity occurs.

As part of an effort to ensure that the JabRef team is focusing on important and valid issues, we would like to ask if you could update the issue if it still persists. This could be in the following form:

  • If there has been a longer discussion, add a short summary of the most important points as a new comment (if not yet existing).
  • Provide further steps or information on how to reproduce this issue.
  • Upvote the initial post if you like to see it implemented soon. Votes are not the only metric that we use to determine the requests that are implemented, however, they do factor into our decision-making process.
  • If all information is provided and still up-to-date, then just add a short comment that the issue is still relevant.

Thank you for your contribution!

@ilippert
Copy link
Contributor Author

ilippert commented Dec 8, 2020

I wonder how difficult this feature is. If it is not much work, then I think it would really help minimise puzzlement for users who are not everyday users of jabref. I repeatedly stumble upon it after some weeks in which I do not search for literature...

@Siedlerchr
Copy link
Member

@DominikVoigt As you have more insight, I think the text of the dialog has been changed, correct?

@ilippert
Copy link
Contributor Author

ilippert commented Dec 8, 2020

JabRef 5.2--2020-12-07--c770310
Linux 5.9.11-200.fc33.x86_64 amd64
Java 15.0.1

just testing it.
When I try to use google search nothing happens. when I type - into the search field, the field turns red.

@github-actions
Copy link
Contributor

This issue has been inactive for half a year. Since JabRef is constantly evolving this issue may not be relevant any longer and it will be closed in two weeks if no further activity occurs.

As part of an effort to ensure that the JabRef team is focusing on important and valid issues, we would like to ask if you could update the issue if it still persists. This could be in the following form:

  • If there has been a longer discussion, add a short summary of the most important points as a new comment (if not yet existing).
  • Provide further steps or information on how to reproduce this issue.
  • Upvote the initial post if you like to see it implemented soon. Votes are not the only metric that we use to determine the requests that are implemented, however, they do factor into our decision-making process.
  • If all information is provided and still up-to-date, then just add a short comment that the issue is still relevant.

Thank you for your contribution!

@ilippert
Copy link
Contributor Author

JabRef 5.3--2021-07-16--c99d61c
Linux 5.13.4-200.fc34.x86_64 amd64
Java 16.0.1
JavaFX 16+8

The field turns red when entering - and a hover tip signals an invalid query element '-' at position X. I think this is a good solution for now. Therefore I close this issue.

koppor pushed a commit that referenced this issue Jan 1, 2023
43566f2 Update molecular-oncology.csl (#6354)
38f2b5f Update san-francisco-estuary-and-watershed-science.csl (#6350)
724cb12 Update australasian-journal-of-philosophy.csl (#6344)
df6af86 Fix webpage in-text citation for council-of-science-editors-author-date.csl (#6318)
6900b58 Add month to magazine for Bluebook
e0a8148 Update cambridge-university-press-author-date-cambridge-a.csl (#6345)
0823448 Add space & comma before pp (#6343)
ff38cd2 Update american-journal-of-respiratory-and-critical-care-medicine.csl (#6341)
8727dfb Update early-music-history.csl (#6323)
1154354 Update boletin-de-pediatria.csl (#6310)
f25438e Update pravnik.csl (#6309)
db7a4ae Update zoological-journal-of-the-linnean-society.csl (#6304)
6c043a7 Create polygraphia.csl (#6307)
255e00c Create intellect-newgen-books.csl (#6308)
323629f Update historical-materialism.csl (#6300)
f62b70d Create european-review-of-international-studies.csl (#6301)
dff2698 Update ucl-university-college-harvard.csl (#6298)
0ce09c9 Update sciences-po-ecole-doctorale-note-french.csl (#6290)
bdd53ec Update sciences-po-ecole-doctorale-author-date.csl (#6291)
efde4d4 Create journal-of-law-medicine-ethics.csl (#6296)
7539b2c Create theses-de-sorbonne-universite.csl (#6295)
905f25a Update biochemical-society-transactions.csl (#6292)
a76a3f5 Update smithsonian-institution-scholarly-press author-date and note (#6294)
e6b6c6c Create exploration-of-targeted-anti-tumor-therapy.csl (#6276)
024c9c8 Create nys-nydanske-studier.csl (#6331)
d9ac8e1 Update vox-sanguinis.csl (#6327)
9a98e92 Remove DOI from Genetics & Molecular Biology

git-subtree-dir: buildres/csl/csl-styles
git-subtree-split: 43566f2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants