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

fix: Use URL for debug bug pattern URL #146

Merged
merged 1 commit into from Mar 15, 2023

Conversation

bdrung
Copy link
Collaborator

@bdrung bdrung commented Mar 15, 2023

bug_pattern_url must be a URL and not only a file. Otherwise apport will fail:

Traceback (most recent call last):
  File "/usr/share/apport/apport-gtk", line 698, in <module>
    app.run_argv()
  File "/usr/lib/python3/dist-packages/apport/ui.py", line 868, in run_argv
    return self.run_report_bug()
  File "/usr/lib/python3/dist-packages/apport/ui.py", line 627, in run_report_bug
    self.collect_info(symptom_script)
  File "/usr/lib/python3/dist-packages/apport/ui.py", line 1624, in collect_info
    bpthread.exc_raise()
  File "/usr/lib/python3/dist-packages/apport/REThread.py", line 62, in exc_raise
    raise self._exception[1].with_traceback(self._exception[2])
  File "/usr/lib/python3/dist-packages/apport/REThread.py", line 37, in run
    self._retval = self.__target(*self.__args, **self.__kwargs)
  File "/usr/lib/python3/dist-packages/apport/report.py", line 1232, in search_bug_patterns
    with urllib.request.urlopen(url) as request:
  File "/usr/lib/python3.10/urllib/request.py", line 216, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python3.10/urllib/request.py", line 503, in open
    req = Request(fullurl, data)
  File "/usr/lib/python3.10/urllib/request.py", line 322, in __init__
    self.full_url = url
  File "/usr/lib/python3.10/urllib/request.py", line 348, in full_url
    self._parse()
  File "/usr/lib/python3.10/urllib/request.py", line 377, in _parse
    raise ValueError("unknown url type: %r" % self.full_url)
ValueError: unknown url type: '/tmp/bugpatterns.xml'

`bug_pattern_url` must be a URL and not only a file. Otherwise apport
will fail:

```
Traceback (most recent call last):
  File "/usr/share/apport/apport-gtk", line 698, in <module>
    app.run_argv()
  File "/usr/lib/python3/dist-packages/apport/ui.py", line 868, in run_argv
    return self.run_report_bug()
  File "/usr/lib/python3/dist-packages/apport/ui.py", line 627, in run_report_bug
    self.collect_info(symptom_script)
  File "/usr/lib/python3/dist-packages/apport/ui.py", line 1624, in collect_info
    bpthread.exc_raise()
  File "/usr/lib/python3/dist-packages/apport/REThread.py", line 62, in exc_raise
    raise self._exception[1].with_traceback(self._exception[2])
  File "/usr/lib/python3/dist-packages/apport/REThread.py", line 37, in run
    self._retval = self.__target(*self.__args, **self.__kwargs)
  File "/usr/lib/python3/dist-packages/apport/report.py", line 1232, in search_bug_patterns
    with urllib.request.urlopen(url) as request:
  File "/usr/lib/python3.10/urllib/request.py", line 216, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python3.10/urllib/request.py", line 503, in open
    req = Request(fullurl, data)
  File "/usr/lib/python3.10/urllib/request.py", line 322, in __init__
    self.full_url = url
  File "/usr/lib/python3.10/urllib/request.py", line 348, in full_url
    self._parse()
  File "/usr/lib/python3.10/urllib/request.py", line 377, in _parse
    raise ValueError("unknown url type: %r" % self.full_url)
ValueError: unknown url type: '/tmp/bugpatterns.xml'
```

Signed-off-by: Benjamin Drung <benjamin.drung@canonical.com>
@bdrung bdrung requested a review from schopin-pro March 15, 2023 10:16
@codecov
Copy link

codecov bot commented Mar 15, 2023

Codecov Report

Merging #146 (ae1f159) into main (eb0acb1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #146   +/-   ##
=======================================
  Coverage   81.95%   81.95%           
=======================================
  Files          87       87           
  Lines       18197    18197           
=======================================
  Hits        14914    14914           
  Misses       3283     3283           
Impacted Files Coverage Δ
etc/apport/crashdb.conf 100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@schopin-pro schopin-pro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@schopin-pro schopin-pro merged commit 22f4525 into canonical:main Mar 15, 2023
24 checks passed
@bdrung bdrung deleted the fix-bug-pattern-url branch March 15, 2023 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants