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 support for experimental image based on Alpine Linux #2252

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

eduardo010174
Copy link
Contributor

  • Image build correctly
  • All modules for python install correctly
  • Error detected on pybluemonday on runtime.
Traceback (most recent call last):
  File "/opt/CTFd/ping.py", line 10, in <module>
    from CTFd.config import Config
  File "/opt/CTFd/CTFd/__init__.py", line 19, in <module>
    from CTFd.plugins import init_plugins
  File "/opt/CTFd/CTFd/plugins/__init__.py", line 9, in <module>
    from CTFd.utils.config.pages import get_pages
  File "/opt/CTFd/CTFd/utils/config/pages.py", line 8, in <module>
    from CTFd.utils.security.sanitize import sanitize_html
  File "/opt/CTFd/CTFd/utils/security/sanitize.py", line 1, in <module>
    from pybluemonday import UGCPolicy
  File "/opt/venv/lib/python3.9/site-packages/pybluemonday/__init__.py", line 5, in <module>
    from pybluemonday.bluemonday import ffi, lib
ImportError: Error relocating /opt/venv/lib/python3.9/site-packages/pybluemonday/bluemonday.cpython-39-x86_64-linux-gnu.so: free: initial-exec TLS resolves to dynamic definition in /opt/venv/lib/python3.9/site-packages/pybluemonday/bluemonday.cpython-39-x86_64-linux-gnu.so

Probably it's this error.
(golang/go#54805)

After I have time, I will open Issue on (https://github.com/ColdHeat/pybluemonday) for track this problem.

@codecov
Copy link

codecov bot commented Jan 31, 2023

Codecov Report

Merging #2252 (d2d64ab) into master (2a6f47d) will not change coverage.
The diff coverage is n/a.

❗ Current head d2d64ab differs from pull request most recent head 4940671. Consider uploading reports for the commit 4940671 to get more accurate results

@@           Coverage Diff           @@
##           master    #2252   +/-   ##
=======================================
  Coverage   87.61%   87.61%           
=======================================
  Files         145      145           
  Lines        8858     8858           
=======================================
  Hits         7761     7761           
  Misses       1097     1097           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

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.

None yet

1 participant