Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

Can we remove this `sed`?

Open
#48 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
45/100
Issue type
Refactor
Clarity
Clearly specified
Activity status
Stale
Tech stack
dockerfile
Domain
devops

Research direction

Start with Dockerfile around line 13 and review the OpenSSL workaround command and its surrounding image setup. Confirm the stated npm.sap.com and Debian Buster assumptions, then verify that the image still builds after the obsolete sed workaround is removed.

Written by the indexing model from the issue text.

Description

It we have the following, here in the dockerfile:

# Workaround for https://npm.sap.com issue with open SSL in Debian Buster
RUN CIPHERS="$(openssl ciphers)" && sed -i "s/DEFAULT@SECLEVEL=2/$CIPHERS:DH-RSA-AES256-SHA256/g" /etc/ssl/openssl.cnf

Now we do not user npm.sap.com anymore and buster is also about to be removed. Can we remove this sed?

@fwilhe Could you comment? Do you have any memory of this?

Dominant language
Dockerfile
Stars
11
Forks
7
PR merge metrics
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from SAP/devops-docker-node-browsers

All issues in SAP/devops-docker-node-browsers

Similar issues

More DevOps issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.