Skip to content

Conversation

margaretjgu
Copy link
Member

No description provided.

@prodsecmachine
Copy link

prodsecmachine commented Aug 26, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

.github/make.sh Outdated
@@ -22,7 +22,7 @@
# ------------------------------------------------------- #
# Bootstrap
# ------------------------------------------------------- #
script_path=$(dirname "$(realpath -s "$0")")
script_path=$(dirname "$(realpath "$0")")
Copy link
Member Author

Choose a reason for hiding this comment

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

realpath: illegal option -- s
usage: realpath [-q] [path ...]

unfortunately realpath -s doesnt not exist for mac but does for linux

RUN (id -g ${BUILDER_GID} || groupadd --system -g ${BUILDER_GID} ${BUILDER_GROUP}) \
&& (id -u ${BUILDER_UID} || useradd --system --shell /bin/bash -u ${BUILDER_UID} -g ${BUILDER_GID} -m elastic) \
RUN (getent group ${BUILDER_GID} || groupadd --system -g ${BUILDER_GID} ${BUILDER_GROUP}) \
&& (getent passwd ${BUILDER_UID} || useradd --system --shell /bin/bash -u ${BUILDER_UID} -g ${BUILDER_GID} -m elastic) \
Copy link
Member Author

Choose a reason for hiding this comment

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

ERROR: failed to solve: process "/bin/sh -c (id -g ${BUILDER_GID} || groupadd --system -g ${BUILDER_GID} ${BUILDER_GROUP})"
------
 > [3/7] RUN (id -g 20 || groupadd --system -g 20 elastic)     && (id -u 501 || useradd --system --shell /bin/bash -u 501 -g 20 -m elastic)     && mkdir -p /usr/src/elasticsearch-js     && chown -R 501:20 /usr/src/:
0.115 id: '20': no such user
0.116 groupadd: GID '20' already exists

use getent to check if group and user id exists, if not then it creates. if exists, then skip creating

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