Phase05 #8
Phase05 #8
Conversation
Co-authored-by: Mohammad Sadegh Montazeri <[email protected]>
|
|
||
| WORKDIR /app | ||
|
|
||
| RUN adduser -D -g '' -u 10001 builder |
There was a problem hiding this comment.
من جایی دیدم که نباید به صورت دستی یک UID مشخص ست کنیم. حالا نمیدونم چطوری هم بالای 10000 باشه و هم دستی ست نشه.
There was a problem hiding this comment.
من مثلا ایمیج gitea rootless رو میبینم uid تنظیم شده و روی 1000 هم تنظیم شده. عجیبه.
@dkhorasanizadeh
شما نظری ندارید؟
| var DomainRegex = regexp.MustCompile(`^(([a-zA-Z0-9]|[a-zA-Z0-9][a-zA-Z0-9\-]*[a-zA-Z0-9])\.)*([A-Za-z0-9]|[A-Za-z0-9][A-Za-z0-9\-]*[A-Za-z0-9])$`) | ||
| var AddrRegex = regexp.MustCompile(`^(([0-9]|[1-9][0-9]|1[0-9]{2}|2[0-4][0-9]|25[0-5])\.){3}([0-9]|[1-9][0-9]|1[0-9]{2}|2[0-4][0-9]|25[0-5])$`) |
There was a problem hiding this comment.
بهتر نبود این دو تا متغیر داخل توابع مشخصی تعریف و همونجا برای چک کردن رجکس استفاده بشن که مجبور نباشید متغیر ها رو گلوبال تعریف کنید؟
There was a problem hiding this comment.
به ۲ دلیل انجام دادم. یکی اینکه هربار regex کامپایل نشه و و فقط موقع startup این کار انجام بشه و یکی دیگه هم اینکه چون از MustCompile استفاده شده اگه regex مشکل داشته باشه موقع startup برنامه crash میکنه و آدم سریع متوجه اشکالش میشه.
There was a problem hiding this comment.
به هیچ وجه مشکلی حساب نمیشه و صرفا برام سوال بود که چرا از gin استفاده نکردید؟ :)
There was a problem hiding this comment.
چون API ای که ازمون میخواستن ساده بود. با هم که صحبت کردیم در نهایت به این نتجه رسیدیم gin لازم نیست و net/http کافیه. جدای از این با این کار بیشتر با golang آشنا شدیم.
…instead of panic.
No description provided.