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

webbrowser: wrong arguments for Opera browser. #78200

Closed
kbumsik mannequin opened this issue Jul 2, 2018 · 12 comments
Closed

webbrowser: wrong arguments for Opera browser. #78200

kbumsik mannequin opened this issue Jul 2, 2018 · 12 comments
Labels
3.7 (EOL) end of life 3.8 (EOL) end of life stdlib Python modules in the Lib dir

Comments

@kbumsik
Copy link
Mannequin

kbumsik mannequin commented Jul 2, 2018

BPO 34019
Nosy @taleinat, @pablogsal, @miss-islington, @kbumsik
PRs
  • bpo-34019: Fix wrong arguments for Opera Browser #8047
  • [3.7] bpo-34019: Fix wrong arguments for Opera Browser (GH-8047) #8063
  • [3.6] bpo-34019: Fix wrong arguments for Opera Browser (GH-8047) #8064
  • [2.7] bpo-34019: Fix wrong arguments for Opera Browser (GH-8047) #8126
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = None
    closed_at = <Date 2018-07-08.05:36:43.344>
    created_at = <Date 2018-07-02.09:51:20.632>
    labels = ['3.8', '3.7', 'library']
    title = 'webbrowser: wrong arguments for Opera browser.'
    updated_at = <Date 2018-07-08.05:36:43.342>
    user = 'https://github.com/kbumsik'

    bugs.python.org fields:

    activity = <Date 2018-07-08.05:36:43.342>
    actor = 'taleinat'
    assignee = 'none'
    closed = True
    closed_date = <Date 2018-07-08.05:36:43.344>
    closer = 'taleinat'
    components = ['Library (Lib)']
    creation = <Date 2018-07-02.09:51:20.632>
    creator = 'kbumsik'
    dependencies = []
    files = []
    hgrepos = []
    issue_num = 34019
    keywords = ['patch']
    message_count = 12.0
    messages = ['320863', '320922', '320923', '320926', '320927', '320928', '320964', '320974', '321077', '321143', '321194', '321257']
    nosy_count = 4.0
    nosy_names = ['taleinat', 'pablogsal', 'miss-islington', 'kbumsik']
    pr_nums = ['8047', '8063', '8064', '8126']
    priority = 'normal'
    resolution = 'fixed'
    stage = 'resolved'
    status = 'closed'
    superseder = None
    type = None
    url = 'https://bugs.python.org/issue34019'
    versions = ['Python 2.7', 'Python 3.6', 'Python 3.7', 'Python 3.8']

    @kbumsik
    Copy link
    Mannequin Author

    kbumsik mannequin commented Jul 2, 2018

    webbrowser module currently opens Opera Browser using the following command:

    opera -remote "openURL(https://google.com,new-window)"

    While this follows Opera's documentation (https://www.opera.com/docs/switches/#remote), this is broken since Opera switched from its own engine to Chromium. The working command is exactly the same as Chrome:

    opera --new-window https://google.com

    @kbumsik kbumsik mannequin added 3.7 (EOL) end of life stdlib Python modules in the Lib dir 3.8 (EOL) end of life labels Jul 2, 2018
    @pablogsal
    Copy link
    Member

    I can use

    opera -remote "openURL(https://google.com,new-window)"

    without problems as per the opera documentation with the latest version:

    $opera --version
    53.0.2907.110

    @pablogsal
    Copy link
    Member

    What error do you get and what opera version are you using?

    @kbumsik
    Copy link
    Mannequin Author

    kbumsik mannequin commented Jul 2, 2018

    $opera --version
    53.0.2907.68

    $opera -remote "openURL(https://google.com,new-window)"

    would open a broken link: http://openurl%28https//google.com,new-window)

    I found that problem while answering a StackOverflow question: https://stackoverflow.com/questions/51123537/webbrowser-open-breaks-the-link-python-3-7

    @pablogsal
    Copy link
    Member

    Sorry, you are right! It seems that my Opera opened another tab with Google by default and I was confused by that. Thanks for the context.

    @kbumsik
    Copy link
    Mannequin Author

    kbumsik mannequin commented Jul 2, 2018

    No problem :) To add more, the Opera's documentation (https://www.opera.com/docs/switches) says:

    "This document was last updated for Opera 11.61"

    This is the reason I guessed Opera changed its CLI format to Chrome's and forget to document it....

    @pablogsal
    Copy link
    Member

    New changeset 3cf1f15 by Pablo Galindo (Bumsik Kim) in branch 'master':
    bpo-34019: Fix wrong arguments for Opera Browser (bpo-8047)
    3cf1f15

    @miss-islington
    Copy link
    Contributor

    New changeset dd5f43a by Miss Islington (bot) in branch '3.6':
    bpo-34019: Fix wrong arguments for Opera Browser (GH-8047)
    dd5f43a

    @taleinat
    Copy link
    Contributor

    taleinat commented Jul 5, 2018

    Shouldn't this be backported to 2.7 as well?

    @kbumsik
    Copy link
    Mannequin Author

    kbumsik mannequin commented Jul 5, 2018

    I also believe this can be backported to 2.7 as well.

    @pablogsal
    Copy link
    Member

    New changeset 8902a1d by Pablo Galindo in branch '2.7':
    [2.7] bpo-34019: Fix wrong arguments for Opera Browser (GH-8047) (bpo-8126)
    8902a1d

    @taleinat
    Copy link
    Contributor

    taleinat commented Jul 8, 2018

    Thanks for reporting the issue and the PR, Bumsik Kim!

    @taleinat taleinat closed this as completed Jul 8, 2018
    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    3.7 (EOL) end of life 3.8 (EOL) end of life stdlib Python modules in the Lib dir
    Projects
    None yet
    Development

    No branches or pull requests

    3 participants