mirror of
https://github.com/Cute-Dress/Dress.git
synced 2026-04-17 06:08:11 +00:00
添加检查exif信息的工作流, 完善文档 (#299)
All checks were successful
Mark stale issues and pull requests / stale (push) Successful in 5s
All checks were successful
Mark stale issues and pull requests / stale (push) Successful in 5s
* 慕名而来玩玩 * 换格式 把webp格式的图片换成jpg了 似乎github那边的pr没办法预览webp * 更新两张新图 * 定期更新一下 * Update README.md * Create README.md with updates and learnings Added a README file with updates and personal reflections. * Add files via upload * 更新嘻嘻 * Initial plan * Add PR template, EXIF check workflow, and strip-exif workflow Co-authored-by: Yueosa <172176062+Yueosa@users.noreply.github.com> * Address code review: reduce exiftool timeout, improve error logging in strip workflow Co-authored-by: Yueosa <172176062+Yueosa@users.noreply.github.com> * Security: pin checkout to SHA, add env vars for shell safety, add security comments Co-authored-by: Yueosa <172176062+Yueosa@users.noreply.github.com> * Improve PR template with newcomer guidance, friendlier EXIF comment, add beginner guide, update README Co-authored-by: Yueosa <172176062+Yueosa@users.noreply.github.com> * Fix PR template links to use absolute GitHub URLs Co-authored-by: Yueosa <172176062+Yueosa@users.noreply.github.com> * 修改示例中的 ID 为我自己的ID * delete(action): 删除没必要的工作流 * fix(action): 删除对strip_exif工作流的描述 * docs(GUIDE): 为 fork, commit 等操作添加了简单说明 * fix(docs): OMG我是猪鼻, 我把仓库地址写错了X_X * fix: 被 copilot 鞭策了 * 删除拉取请求模板中的描述部分 * 补充推荐使用--depth的妙妙小提示 * - 补充删旧评论机制 - 移除不必要的权限 - 只在图片变更时触发 - 修复fork pr拿不到来源的bug * 优化拉取请求模板,增加新手指南的可折叠提示 * 更新 check_exif.yml:升级 actions/checkout 到 v5,优化 exiftool 安装逻辑,增强 PR 评论管理 * 到底要不要删掉旧评论呢(哭), Moemu大佬好像是没删的, 豪! 我也不删了! * 被删除的不必要的权限似乎是必要的 * 修一修代码格式, 压一压行数 * 修正评论格式 --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Yueosa <172176062+Yueosa@users.noreply.github.com>
This commit is contained in:
105
.github/workflows/check_exif.yml
vendored
Normal file
105
.github/workflows/check_exif.yml
vendored
Normal file
@@ -0,0 +1,105 @@
|
||||
name: Check Image EXIF Data
|
||||
|
||||
on:
|
||||
pull_request_target:
|
||||
types: [opened, synchronize, reopened]
|
||||
paths: ['**/*.jpg', '**/*.jpeg', '**/*.png', '**/*.webp', '**/*.tiff', '**/*.tif', '**/*.heic', '**/*.heif', '**/*.avif', '**/*.gif']
|
||||
|
||||
permissions:
|
||||
contents: read
|
||||
pull-requests: write
|
||||
issues: write
|
||||
|
||||
concurrency:
|
||||
group: exif-pr-${{ github.event.pull_request.number }}
|
||||
cancel-in-progress: true
|
||||
|
||||
jobs:
|
||||
check-exif:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: Install exiftool
|
||||
run: |
|
||||
command -v exiftool && exit 0
|
||||
sudo apt-get install -y --no-install-recommends libimage-exiftool-perl \
|
||||
|| (sudo apt-get update && sudo apt-get install -y --no-install-recommends libimage-exiftool-perl)
|
||||
|
||||
- name: Check EXIF in changed images
|
||||
uses: actions/github-script@v8
|
||||
with:
|
||||
script: |
|
||||
const { execSync } = require('child_process');
|
||||
const fs = require('fs');
|
||||
const path = require('path');
|
||||
const os = require('os');
|
||||
const pr = context.payload.pull_request;
|
||||
const { owner, repo } = context.repo;
|
||||
const prNumber = pr.number;
|
||||
const IMAGE_EXT = /\.(jpe?g|png|webp|tiff?|hei[cf]|avif|gif)$/i;
|
||||
|
||||
// 1. Collect changed image files
|
||||
let changedImages = [], page = 1;
|
||||
while (true) {
|
||||
const { data: files } = await github.rest.pulls.listFiles({ owner, repo, pull_number: prNumber, per_page: 100, page });
|
||||
if (!files.length) break;
|
||||
for (const f of files) {
|
||||
if (['added', 'modified', 'renamed', 'copied'].includes(f.status) && IMAGE_EXT.test(f.filename))
|
||||
changedImages.push(f.filename);
|
||||
}
|
||||
if (files.length < 100) break;
|
||||
page++;
|
||||
}
|
||||
if (!changedImages.length) { core.info('No changed images.'); return; }
|
||||
core.info(`Checking ${changedImages.length} image(s) for EXIF data.`);
|
||||
|
||||
// 2. Download & check each image
|
||||
const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'exif-'));
|
||||
const exifFiles = [], errors = [];
|
||||
|
||||
for (const file of changedImages) {
|
||||
const tmpFile = path.join(tmp, file.replace(/[\\/]/g, '__'));
|
||||
try {
|
||||
const { data } = await github.rest.repos.getContent({
|
||||
owner: pr.head.repo.owner.login, repo: pr.head.repo.name,
|
||||
path: file, ref: pr.head.sha
|
||||
});
|
||||
let raw;
|
||||
if (!Array.isArray(data) && data.content) raw = Buffer.from(data.content, 'base64');
|
||||
else if (!Array.isArray(data) && data.download_url) {
|
||||
const resp = await fetch(data.download_url);
|
||||
if (!resp.ok) throw new Error(`Download ${resp.status}`);
|
||||
raw = Buffer.from(await resp.arrayBuffer());
|
||||
} else throw new Error('Cannot fetch file content');
|
||||
|
||||
fs.writeFileSync(tmpFile, raw);
|
||||
const count = parseInt(execSync(
|
||||
`exiftool -EXIF:all -S -q -- '${tmpFile.replace(/'/g, "'\\''")}' 2>/dev/null | wc -l`,
|
||||
{ encoding: 'utf-8', timeout: 10000 }
|
||||
).trim()) || 0;
|
||||
if (count > 0) { exifFiles.push(file); core.info(`EXIF found: ${file} (${count} tags)`); }
|
||||
} catch (e) { core.warning(`Check failed: ${file}: ${e.message}`); errors.push(file); }
|
||||
finally { try { fs.unlinkSync(tmpFile); } catch {} }
|
||||
}
|
||||
try { fs.rmdirSync(tmp); } catch {}
|
||||
|
||||
// 3. Report results
|
||||
if (errors.length) {
|
||||
core.setFailed(`EXIF check failed for: ${errors.join(', ')}. Please retry or ask a maintainer.`);
|
||||
return;
|
||||
}
|
||||
if (exifFiles.length) {
|
||||
const fileList = exifFiles.map(f => `- ${f}`).join('\n');
|
||||
const body = [
|
||||
'我们检测到以下文件包含 EXIF 信息,请移除 EXIF 数据后再次提交:',
|
||||
'We detected EXIF data in the following files. Please remove the EXIF data and resubmit:',
|
||||
'',
|
||||
fileList,
|
||||
'',
|
||||
`📖 [CONTRIBUTING.md](https://github.com/${owner}/${repo}/blob/master/CONTRIBUTING.md)`
|
||||
].join('\n');
|
||||
try { await github.rest.issues.createComment({ owner, repo, issue_number: prNumber, body }); }
|
||||
catch (e) { core.warning(`Cannot comment: ${e.message}. Check repo Settings > Actions > Workflow permissions.`); }
|
||||
core.setFailed('EXIF data detected. Please remove EXIF data for privacy protection.');
|
||||
} else {
|
||||
core.info('No EXIF data found. ✅');
|
||||
}
|
||||
Reference in New Issue
Block a user