Comments (6)
Seems like,
https://github.com/cli/browser/blob/419c10c56bd56293c8f898ef5c6be291eb099120/browser_linux.go#L8
here,
providers := []string{"xdg-open", "x-www-browser", "www-browser", "wslview"}
provider is found but, XDG_SESSION_TYPE is tty, while watching the os env list
I can send a PR for this if accepted as a BUG.
from go-gh.
@leodahal4 Thanks for writing in with your issue. This sounds more like an incorrect local environment variable configuration than an issue with the package. What would your PR to fix this bug include?
from go-gh.
I have tried to check the os env variables and if the session's env contains XDG_SESSION_TYPE as tty then we can fail returning error as cannot execute the browser, and its working fine on mine CLI as without browser user can visit a predefined webpage. But seems like I should send the PR to https://github.com/cli/browser, right?
from go-gh.
This was failing for me recently and it was a WSL bug. Does invoking wslview
manually work for you?
from go-gh.
no unfortunately as I am on arch Linux, and my environment doesnot include wslview
from go-gh.
@leodahal4 Thanks for following up. We have discussed this a bit further and we feel that checking XDG_SESSION_TYPE
environment variable is a bit of an overstep and does not necessarily indicate that invoking xdg-open
will fail. I am going to close this as won't fix as I don't see anything we can do from our side. Please let me know if you feel otherwise.
from go-gh.
Related Issues (20)
- Support for stdin and stderr in term package
- GQLCLient should return GQLError for query and mutation methods
- Add a GraphQL Mutation with an `input` object to examples HOT 1
- Add pagination examples
- jsonpretty should not add new lines if not indenting
- TokenForHost "authentication token not found" message could be more informative HOT 3
- Feature Request: Add support to call the `gh` command interactively HOT 3
- RestClient no longer returns an HTTPError HOT 2
- Expose `ghLookupPath` or a better way to run `gh` HOT 3
- `asciisanitizer.Sanitizer` mishandled the `�` unicode character HOT 1
- Add the ability to introduce repository specific configurations for gh-cli and its extensions HOT 2
- Create CODEOWNERS file ensuring the cli/code-reviewers are assigned to PRs
- CVE Vulnerability CVE-2023-3978 in dependency HOT 1
- hyperlink truncated at end of row isn't closed correctly HOT 2
- Use retryable HTTP client (for 50x) errors
- Allow default host to be determined based on configuration preference HOT 3
- Rest client support for endpoint not returning a JSON HOT 3
- AlecAivazis/survey is archived HOT 3
- Support passing httptest URLs in api.ClientOptions HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from go-gh.