oss-gate / workshop

OSSの開発に未参加または参加したことはあるけどまだ自信がない人を後押しするワークショップ用のリポジトリー
124 stars 547 forks source link

OSS Gate Workshop: Tokyo: 2024-09-21: shallowfield: Athenz: Work log #1876

Closed shallowfield closed 6 days ago

shallowfield commented 6 days ago

This is a work log of a "OSS Gate workshop". "OSS Gate workshop" is an activity to increase OSS developers. Here's been discussed in Japanese. Thanks.

作業ログ作成時の説明

以下のテンプレートを埋めてタイトルに設定します。埋め方例はスクロールすると見えてきます。

OSS Gate Workshop: ${LOCATION}: ${YEAR}-${MONTH}-${DAY}: ${ACCOUNT_NAME}: ${OSS_NAME}: Work log

タイトル例↓:

OSS Gate Workshop: Tokyo: 2017-01-16: kou: Rabbit: Work log

OSS Gateワークショップ関連情報

shallowfield commented 6 days ago

今日は何を触ろう

shallowfield commented 6 days ago

https://github.com/AthenZ/athenz のutilがWindowsに対応していないのが許せない

shallowfield commented 6 days ago

https://github.com/AthenZ/athenz/blob/master/LICENSE

shallowfield commented 6 days ago

何故Home Directoryを取るのに標準ライブラリを使わないのか? https://github.com/AthenZ/athenz/blob/master/utils/zms-cli/zms-cli.go#L84

shallowfield commented 6 days ago

環境変数周りが怪しい https://github.com/AthenZ/athenz/blob/master/utils/zms-cli/zms-cli.go#L48

shallowfield commented 6 days ago

会社のPCを使っているからgitの設定が会社のアカウントになっているかも知れないので直す

shallowfield commented 6 days ago

良く分からないのでtestリポジトリを作った https://github.com/shallowfield/test

shallowfield commented 6 days ago

https://github.com/shallowfield/test 認証をしました

shallowfield commented 6 days ago

CONTRIBUTINGを見る https://github.com/AthenZ/athenz/blob/master/CONTRIBUTING.md

shallowfield commented 6 days ago

やはり先にissueを立てた方が良さそう。

内容

shallowfield commented 6 days ago

コード部分を見ています。 userとhomedirは分かるのですが、他の環境変数の扱いが良く分かっていない

shallowfield commented 6 days ago

Linuxは常にSOCKS5_PROXYの中身があるのか?

blacktoad30 commented 6 days ago

https://github.com/cloudfoundry/socks5-proxy/blob/main/socks5_proxy.go SOCKS5_PROXY で検索したら、上に出てきた

shallowfield commented 6 days ago

気にしている部分はこれ https://github.com/AthenZ/athenz/blob/master/utils/zms-cli/zms-cli.go#L52 https://linuxtutorials.org/socks5-proxy-environment-variable-linux/

shallowfield commented 6 days ago

とりあえずユーザー名とHomeDirだけ修正してPRを投げる どんな雰囲気のPRがあるのか確認する

shallowfield commented 6 days ago

過去の修正を見ていると、ユーザー名関連で更に修正が入っているので確認しています

shallowfield commented 6 days ago

社内事例でやはりUSER環境変数を参照したい場合があるということで追加の修正がなされていた。詳細が分からないので対応方針が分からない。

shallowfield commented 6 days ago

issueを上げるに留めておきます。 タイトル request for multi-platform support for cli tools

shallowfield commented 6 days ago

next action

github-actions[bot] commented 6 days ago

おつかれさまでした!

ワークショップの終了にともないissueを閉じますが、このまま作業メモとして使っても構いません :ok_hand:

ワークショップの感想を集めています!

ブログなどに書かれた際は、このページへリンクの追加をお願いします :pray:

またの参加をお待ちしています!