Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

Fix the issue that restoring NuGet packages failed on mac-latest image. #1045

Merged
merged 5 commits into from
Jun 24, 2022

Conversation

keiji
Copy link
Collaborator

@keiji keiji commented Jun 18, 2022

Issue 番号 / Issue ID

目的 / Purpose

  • GitHub ActionsでiOSビルドが失敗する問題のワークアラウンド

変更内容 / Changes

  • NuGet.Configをセットアップするステップを一時的に追加する

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other... Please describe: workaround

確認事項 / What to check

  • [ ]

その他 / Other information


Internal IDs:

@keiji keiji force-pushed the fix_github_actions_app_build_ios branch 4 times, most recently from 08b9bdd to db706fb Compare June 18, 2022 04:59
@keiji keiji force-pushed the fix_github_actions_app_build_ios branch from db706fb to c0a013c Compare June 18, 2022 05:03
@keiji keiji self-assigned this Jun 18, 2022
@keiji keiji changed the title Specify container image version. Fix the issue that restoring NuGet packages failed on mac-latest image. Jun 18, 2022
@keiji keiji marked this pull request as ready for review June 18, 2022 05:17
@daisuke-nogami
Copy link
Collaborator

daisuke-nogami commented Jun 23, 2022

reviewの観点メモ

  • issueの意図「NugetConfigに直接nuget.orgを登録する」と一致するか
  • Checkで問題あるエラーが起きていないか
    • SonarCloudのtokenが設定されていないというエラーは問題はないのか⇒tokenが見える場所に置いてある方が望ましくないだろう

Copy link
Collaborator

@daisuke-nogami daisuke-nogami left a comment

Choose a reason for hiding this comment

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

チェックの観点に沿って問題がないことを確認しました

@keiji keiji merged commit 4034b79 into cocoa-mhlw:develop Jun 24, 2022
@ghost
Copy link

ghost commented Jun 24, 2022

@keiji , @daisuke-nogami
開発チーム側で使用しているAzure DevOpsのPipelinesでエラーが発生する状態になってしまいました。
一旦切り戻しが必要かもしれません。

Starting: Secure Supply Chain Analysis (auto-injected by policy)
==============================================================================
Task         : Secure Supply Chain Analysis
Description  : A task to scan for vulnerabilities in your software supply chain. Formerly "NuGet Security Analysis".
Version      : 0.2.116
Author       : Microsoft Corporation
Help         : See https://aka.ms/nugetsecurityanalysis for more information. Please contact OpenSourceEngSupport@microsoft.com if you run into problems or have questions with this task.
==============================================================================
Telemetry ID: c9b73f8a-3346-41b1-aadc-ce91b5cb14dc
For more information please visit: https://aka.ms/nugetsecurityanalysis
Starting Multifeed Nuget Security Analysis: 
Starting Multifeed Corext Analysis
Starting Multifeed Python Security Analysis
Starting CFS NuGet Analysis
Starting CFS NPM Analysis
Starting DockerFile Analysis
Azure Artifacts Configuration Analysis found 1 vulnerable NuGet package manifest in the repository. Visit http://aka.ms/azure-artifacts-configuration-analysis for more details.
##[error]NuGet Security Analysis found 1 vulnerable package manifest in the repository. Visit https://aka.ms/nugetmultifeed for more details.
Finishing: Secure Supply Chain Analysis (auto-injected by policy)

@keiji
Copy link
Collaborator Author

keiji commented Jun 24, 2022

DevOpsの方はDevOpsNuget.configのファイル使っているという理解でいて、こちらにも今回追加した同様の記述(<add key="nuget.org" value="https://api.nuget.org/v3/index.json" protocolVersion="3" />)があります。

切り戻しより、なぜvulnerable NuGet package manifestと言われているのか(そしてDevOpsNuget.configの方は当該エラーが起きないのか)。について確認する必要があると思います。

@ghost
Copy link

ghost commented Jun 24, 2022

内部事情ですがテストを実施するのにパイプラインで生成されたものを使用しています。
そのため一部作業のブロック要因となってしまいます。

原因がわかればそれに越したことはありませんが、この事情も踏まえご検討いただければ助かります。

@keiji
Copy link
Collaborator Author

keiji commented Jun 24, 2022

ちょっと調べてみましたが……

https://developercommunity.visualstudio.com/t/nuget-security-analysis-found-1-vulnerable-package/1661039

Thank you for your feedback. We have determined that it’s not a bug.

説明不足も甚だしい上に、ややこしそうだぞ?

内部事情ですがテストを実施するのにパイプラインで生成されたものを使用しています。
そのため一部作業のブロック要因となってしまいます。

別の解決方法もあるので、一回元に戻してそちらのPR作ります!(そちらはNuget.Config触らない)

@keiji keiji deleted the fix_github_actions_app_build_ios branch June 24, 2022 03:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants