Public/pull-request-action
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Compare commits

base: Public:1.0.19
Branches Tags
Public:master Public:add/retry Public:v1 Public:add/failure-verbosity Public:update/yaml-syntax
Public:1.1.1 Public:1.1.0 Public:1.0.24 Public:1.0.23 Public:1.0.22 Public:1.0.21 Public:1.0.20 Public:1.0.19 Public:1.0.18 Public:1.0.17 Public:1.0.16 Public:1.0.15 Public:1.0.14 Public:1.0.13 Public:1.0.12 Public:1.0.11 Public:1.0.10 Public:1.0.2 Public:1.0.1 Public:1.0.0
..
compare: Public:1.0.21
Branches Tags
Public:master Public:add/retry Public:v1 Public:add/failure-verbosity Public:update/yaml-syntax
Public:1.1.1 Public:1.1.0 Public:1.0.24 Public:1.0.23 Public:1.0.22 Public:1.0.21 Public:1.0.20 Public:1.0.19 Public:1.0.18 Public:1.0.17 Public:1.0.16 Public:1.0.15 Public:1.0.14 Public:1.0.13 Public:1.0.12 Public:1.0.11 Public:1.0.10 Public:1.0.2 Public:1.0.1 Public:1.0.0

4 Commits
1.0.19 ... 1.0.21

Author SHA1 Message Date
Yunqi(Alice)
d703f40f3a Update the list_pull_request 401 case (#89) 
* Update the list_pull_request 401 case
Hi, I find this http request is also missing the 401 case.
* Update CHANGELOG.md
 2022-09-28 10:03:05 -06:00
Vanessasaurus
415544869c Merge pull request #88 from ohenak/fix/ghe-get-401-error 
[Chore] Provide token when getting 401 error
 2022-09-22 03:59:00 -06:00
Kane Ho
667293445b [Chore] Provide token when getting 401 error 
Issue #87

In GHE, token is necessary when querying the repo API,
When it fails, it returns 401 error. Will need to specify the token in
this case.
 2022-09-21 23:44:03 -07:00
Felix Breidenstein
e64deeba68 Fix example in README (#85) 2022-03-18 08:44:13 -06:00
3 changed files with 6 additions and 5 deletions
Expand all files Collapse all files

1
CHANGELOG.md
View File

@@ -14,6 +14,7 @@ represented by the pull requests that fixed them. Critical items to know are:
Versions correspond with GitHub releases that can be referenced with @ using actions.
## [master](https://github.com/vsoch/pull-request-action/tree/master) (master)
- bugfix of token handling if 401 error received (missing 401 case) (1.0.21)
- bugfix of writing to environment file (missing newline) (1.0.19)
- bugfix of missing from branch with scheduled run (1.0.16)
- forgot to add assignees (1.0.15)

2
README.md
View File

@@ -119,7 +119,7 @@ jobs:
run: |
# do custom parsing of your code / date to derive a branch from
PR_BRANCH_FROM=release-v$(cat VERSION)
export "PULL_REQUEST_FROM_BRANCH=${PR_BRANCH_FROM}" >> $GITHUB_ENV
echo "PULL_REQUEST_FROM_BRANCH=${PR_BRANCH_FROM}" >> $GITHUB_ENV
- name: pull-request-action
uses: vsoch/pull-request-action@master
env:

8
pull-request.py
View File

@@ -180,8 +180,8 @@ def list_pull_requests(target, source):
print("Params for checking if pull request exists: %s" % params)
response = requests.get(PULLS_URL, params=params)
# Case 1: 404 might warrant needing a token
if response.status_code == 404:
# Case 1: 401, 404 might warrant needing a token
if response.status_code in [401, 404]:
response = requests.get(PULLS_URL, params=params, headers=HEADERS)
if response.status_code != 200:
abort_if_fail(response, "Unable to retrieve information about pull requests")
@@ -238,8 +238,8 @@ def find_default_branch():
"""Find default branch for a repo (only called if branch not provided)"""
response = requests.get(REPO_URL)
# Case 1: 404 might need a token
if response.status_code == 404:
# Case 1: 401, 404 might need a token
if response.status_code in [401, 404]:
response = requests.get(REPO_URL, headers=HEADERS)
if response.status_code != 200:
abort_if_fail(response, "Unable to retrieve default branch")