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

Fix outdated ruby version in Dockerfile #1722

Merged
merged 3 commits into from
Jul 29, 2024

Conversation

ThisIsMissEm
Copy link
Contributor

Summary

This is the Dockerfile changes I needed in #1721 to get the Dockerfile to build on the latest Docker for Mac (and likely other docker versions)

It seems the Dockerfile is still using ruby 2.6, when the project dropped 2.6 support 19 months ago in #1622

I've gone straight ahead to Ruby 3.3.4 (latest stable), despite Doorkeeper not yet officially supporting Ruby 3.3 (I'll open a separate PR to add that to the testing matrix).

Copy link

guardrails bot commented Jul 27, 2024

⚠️ We detected 1 security issue in this pull request:

Insecure Access Control (1)
Severity Details Docs
High Title: Missing User Instruction
FROM ruby:3.3.4-alpine
📚

More info on how to fix Insecure Access Control in Dockerfile.


👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

@ThisIsMissEm
Copy link
Contributor Author

Added ruby 3.3 support to the testing matrix here: #1723

@ThisIsMissEm
Copy link
Contributor Author

This issue should now be fixed: #1722 (comment)

Copy link
Member

@nbulaj nbulaj left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! 👍

@nbulaj nbulaj merged commit e10ce5d into doorkeeper-gem:main Jul 29, 2024
18 checks passed
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.

2 participants