Skip to content

Conversation

@tsusdere
Copy link
Contributor

@tsusdere tsusdere commented Sep 6, 2024

In the upload-artifact action it excludes hidden files that could lead to sensitive data. After talking with the actions team, even though we utilize this action at the very end we upload a .tar file

path: ${{ runner.temp }}/artifact.tar
. Hence, we need to exclude these files when we create the .tar.

@tsusdere tsusdere requested a review from a team as a code owner September 6, 2024 20:38
run: if [ $(find artifact2 -type l | wc -l) != 0 ]; then echo "Symlinks found"; exit 1; fi
shell: bash

- name: Check for absence of hidden files
Copy link
Contributor

Choose a reason for hiding this comment

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

ln -s hello.txt bonjour.txt

addtional change required ^ to generate hidden file, otherwise it test nothing :)

@tsusdere tsusdere requested a review from YiMysty September 9, 2024 20:35
@tsusdere tsusdere enabled auto-merge September 11, 2024 17:43
@tsusdere tsusdere merged commit efaad07 into main Sep 11, 2024
@tsusdere tsusdere deleted the hidden-files branch September 11, 2024 18:09
@douyixuan
Copy link

Sorry, but how should I upload the hidden files which is essential?

@tsusdere
Copy link
Contributor Author

@douyixuan For now if you opt to use this Action hidden files will be ignored. If you require these hidden files to be part of your project you may opt to not use this Action and instead utilize a different method to upload your files https://github.com/actions/upload-pages-artifact?tab=readme-ov-file#artifact-validation.

@jakebailey
Copy link

I just pulled this in via v4.0.0 without noticing that this was changed, and my site lost its .well-known dir, which is used for a lot of different stuff out there (keybase, atproto domain verification, etc).

It seems pretty retrictive to not provide any way to do this other than to manually create tarballs. Can an option be provided to control this? Or at least emit some sort of warning when a file was omitted so people are aware that this happened?

@Mr0grog
Copy link

Mr0grog commented Aug 25, 2025

I definitely appreciate the security-related goal here, but I have to second @jakebailey’s comment. .well-known is a really common and important use of dotfiles and it would be nice to at least see that supported by default (I have not upgraded to v4 yet because I need that!). Ideally, though, it seems reasonable to let people configure a list of paths/globs to always include rather than kicking people off the happy path of this action entirely if they need to include some dotfiles.

@jakebailey
Copy link

I opened #129 to discuss outside a PR 😄

qryxip added a commit to VOICEVOX/voicevox_core that referenced this pull request Sep 16, 2025
actions/upload-pages-artifact v4に入ってくる
actions/upload-pages-artifact#102 により、多分次のファイルがGitHub Pages
に含まれなくなる。それ以外は特に何もないはず。

- /voicevox_core/apis/python_api/.doctrees/**
- /voicevox_core/apis/python_api/.buildinfo
- /voicevox_core/apis/rust_api/.lock
LiamMorrow added a commit to LiamMorrow/LiftLog that referenced this pull request Sep 17, 2025
They included a change which ignores all hidden  files, just based off if it starts with a .

We obviously want to publish a .well-known dir.

See: actions/upload-pages-artifact#129

and: actions/upload-pages-artifact#102
LiamMorrow added a commit to LiamMorrow/LiftLog that referenced this pull request Sep 17, 2025
The recent update to v4 included a change which ignores all hidden  files

We obviously want to publish a .well-known dir, so for now we can just downgrade.

See: actions/upload-pages-artifact#129

and: actions/upload-pages-artifact#102
seanh added a commit to getpelican/pelican that referenced this pull request Oct 4, 2025
In upload-pages-artifact@v4 the maintainers have made the curious
decision to omit hidden files (files whose names begin with ".") from
the site that gets deployed to GitHub Pages and to provide no way to
include a hidden file (at least "for now"), even though static sites may
of course want to include files whose names begin with "."

actions/upload-pages-artifact#102

Work around this in by creating the "artifact" (tar file) ourselves and
uploading it using the upload-artifact action, as suggested in the
upload-pages-artifact PR linked above.
seanh added a commit to getpelican/pelican that referenced this pull request Oct 4, 2025
In upload-pages-artifact@v4 the maintainers have made the curious
decision to omit hidden files (files whose names begin with ".") from
the site that gets deployed to GitHub Pages and to provide no way to
include a hidden file (at least "for now"), even though static sites may
of course want to include files whose names begin with "."

actions/upload-pages-artifact#102

Work around this in by creating the "artifact" (tar file) ourselves and
uploading it using the upload-artifact action, as suggested in the
upload-pages-artifact PR linked above.
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.

6 participants