GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-105983: add incognito mode to webbrowser module by AmirNakhaeii · Pull Request #106069 · python/cpython

Conversation

@AmirNakhaeii

@AmirNakhaeii AmirNakhaeii commented Jun 24, 2023 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Add incognito / private mode to webbrowser module

open browser in incognito mode when in open() function new parameter is equal 3

  • Issue: Support private mode in webbrowser #105983

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
AmirNakhaeii added 2 commits June 24, 2023 17:40
@AmirNakhaeii
pythongh-105983: added incognito / private mode to webbrowser.py
2b9a280
@AmirNakhaeii
pythongh-105983: Fixed some logical errors
545784d
@bedevere-bot bedevere-bot added the awaiting review label Jun 24, 2023
@bedevere-bot

bedevere-bot commented Jun 24, 2023

Copy link
Copy Markdown

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ghost

ghost commented Jun 24, 2023 •
edited by ghost
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.
CLA signed

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

blurb-it Bot and others added 2 commits June 24, 2023 23:51
@blurb-it
📜🤖 Added by blurb_it.
f1a20fb
@AmirNakhaeii
Merge branch 'python:main' into fix-issue-105983
ae0602b
@AmirNakhaeii AmirNakhaeii changed the title Fix issue 105983 gh-105983: add incognito mode to webbrowser module Jun 25, 2023
@bedevere-bot bedevere-bot mentioned this pull request Jun 23, 2023
Support private mode in webbrowser #105983
Closed
@AmirNakhaeii
Merge branch 'main' into fix-issue-105983
9312844
@Eclips4

Eclips4 commented Jun 25, 2023 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

Duplicate of #105984
@kr-g already working on this issue, so, please, close this PR.
You can pick-up another issue, there's a lot of them =)

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@AmirNakhaeii AmirNakhaeii closed this Jun 25, 2023
@AmirNakhaeii AmirNakhaeii reopened this Jun 25, 2023
@AmirNakhaeii

AmirNakhaeii commented Jun 25, 2023

Copy link
Copy Markdown
Author

@Eclips4
Well, what's the problem? I also tried to add this feature.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@AmirNakhaeii
pythongh-105983: Fix incognito flag for edge browser
9e9811b
@kr-g

kr-g commented Jun 25, 2023 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

good day @AmirNakhaeii,

just received your message via email.
and saw the message from @Eclips4

when you would please have a look at the history of the initial pull request #105984 and the code given there you will see that i already did the development before i have created the issue #105983 regarding the new feature. it is a bit confusing sometimes how github organize the flow of information.

anyway i saw that you added also support for edge.

thats great because i dont have microsoft windows.
i do development on linux only.
thank you.

probably best would be that you wait until my pull request is merged, and then you create an own issue regarding the enhancement and support for edge. and an extra pull request for that.

myself would prefer this approach since i can not spent time for taking part in a development where i dont have the proper OS and can not even test it.

thanks for your contribution, and understanding.

ps. do close your pull request #106069, also because you followed the naming convention and gave it the same name as mine - thats might be confusing to others -

AmirNakhaeii reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@kr-g

kr-g commented Jun 25, 2023

Copy link
Copy Markdown

just corrected the different pull requests numbers in my former reply. sorry for the confusion.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@kr-g

kr-g commented Jun 25, 2023 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

@AmirNakhaeii

probably best would be that you wait until my pull request is merged, and then you create an own issue regarding the enhancement and support for edge. and an extra pull request for that.

i dont know if it is possible to create "depening" issues in github.

but probably it might be also an idea to clone my git repo and checkout the branch https://github.com/kr-g/cpython/tree/kr-g-browser-incognito and create from this HEAD ref a new branch where you put your code for edge browser.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@kr-g

kr-g commented Jun 25, 2023

Copy link
Copy Markdown

@AmirNakhaeii

please have also a look here https://github.com/python/cpython/pull/105984/files at the test cases -> Lib/test/test_webbrowser.py

you need to create also one for Edge.
Thanks

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@AmirNakhaeii

AmirNakhaeii commented Jun 26, 2023

Copy link
Copy Markdown
Author

@kr-g
Thank you for your guidance. I will definitely fulfill your request. Good luck.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@AmirNakhaeii AmirNakhaeii closed this Jun 26, 2023
@AmirNakhaeii

AmirNakhaeii commented Jun 26, 2023

Copy link
Copy Markdown
Author

@kr-g
I rarely visit my Gmail. That's why I didn't see your message.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@kr-g kr-g mentioned this pull request Jun 26, 2023
gh-105983: Add private mode support to webbrowser #105984
Closed
@kr-g

kr-g commented Jun 26, 2023

Copy link
Copy Markdown

@AmirNakhaeii

I rarely visit my Gmail. That's why I didn't see your message.

ok fine, i try to keep that in mind

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

awaiting review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@AmirNakhaeii @bedevere-bot @Eclips4 @kr-g
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.