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

Add /auth/legacy-exchange #112

Merged
merged 4 commits into from
Oct 2, 2023
Merged

Conversation

chrisburr
Copy link
Member

@chrisburr chrisburr commented Sep 27, 2023

@chrisburr
Copy link
Member Author

Looks like it breaks in the way we expected at least 🤞

500 Server Error: Internal Server Error for url: https://server:8443/Framework/TornadoProxyManager: <html><title>500: Internal Server Error</title><body>500: Internal Server Error</body></html>

@chrisburr chrisburr force-pushed the legacy-exchange branch 3 times, most recently from a9d50e9 to afbd9a6 Compare September 27, 2023 16:03
Comment on lines 45 to 50
- name: diracx logs
if: ${{ failure() }}
run: docker logs diracx
- name: DIRAC logs
if: ${{ failure() }}
run: cd /tmp/DIRACRepo && ./integration_tests.py logs --no-follow --lines 1000
Copy link
Member Author

Choose a reason for hiding this comment

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

@chrisburr chrisburr force-pushed the legacy-exchange branch 6 times, most recently from 6340c92 to eacb66d Compare September 29, 2023 04:24
@@ -30,7 +30,7 @@ jobs:
- name: Prepare environment
run: |
pip install typer pyyaml gitpython packaging
git clone https://github.com/DIRACGrid/DIRAC.git -b "${{ matrix.dirac-branch }}" /tmp/DIRACRepo
git clone https://github.com/chaen/DIRAC.git -b "diracx_sandbox" /tmp/DIRACRepo
Copy link
Member Author

Choose a reason for hiding this comment

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

@chaen This is also ready for review modulo this change which will be reverted before merging

Copy link
Contributor

Choose a reason for hiding this comment

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

👍 I've added just the 2 minor fixes we needed (location for write_credentials and DiracClient taking a Pref object)

@chrisburr chrisburr merged commit ae3faf7 into DIRACGrid:main Oct 2, 2023
8 checks passed
@chaen chaen linked an issue Oct 17, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add API key -> token exchange route
2 participants