{"id":479265,"date":"2026-05-11T05:57:51","date_gmt":"2026-05-11T05:57:51","guid":{"rendered":"https:\/\/savepearlharbor.com\/?p=479265"},"modified":"-0001-11-30T00:00:00","modified_gmt":"-0001-11-29T21:00:00","slug":"","status":"publish","type":"post","link":"https:\/\/savepearlharbor.com\/?p=479265","title":{"rendered":"Code Review Horror Stories. \u0427\u0430\u0441\u0442\u044c 2: API, \u043e\u0448\u0438\u0431\u043a\u0438 \u0438 graceful shutdown"},"content":{"rendered":"
\n

\u041f\u0440\u043e\u0434\u043e\u043b\u0436\u0435\u043d\u0438\u0435 \u0440\u0430\u0437\u0431\u043e\u0440\u0430 \u0440\u0435\u0430\u043b\u044c\u043d\u043e\u0433\u043e \u043a\u043e\u0434\u0430 \u0441 \u0441\u043e\u0431\u0435\u0441\u0435\u0434\u043e\u0432\u0430\u043d\u0438\u044f. \u0412 <\/em>\u043f\u0435\u0440\u0432\u043e\u0439 \u0447\u0430\u0441\u0442\u0438<\/em><\/a> \u0440\u0430\u0437\u043e\u0431\u0440\u0430\u043b\u0438 8 \u043f\u0440\u043e\u0431\u043b\u0435\u043c concurrency \u0438 memory: race conditions, \u0443\u0442\u0435\u0447\u043a\u0438 \u0433\u043e\u0440\u0443\u0442\u0438\u043d, \u043f\u0440\u043e\u0438\u0433\u043d\u043e\u0440\u0438\u0440\u043e\u0432\u0430\u043d\u043d\u044b\u0439 mutex, TOCTOU. \u042d\u0442\u043e \u0431\u044b\u043b\u0430 \u043f\u0435\u0440\u0432\u0430\u044f \u043f\u043e\u043b\u043e\u0432\u0438\u043d\u0430 \u0438\u0437 21 \u0431\u0430\u0433\u0430 \u0432 \u043e\u0434\u043d\u043e\u043c \u0441\u0435\u0440\u0432\u0438\u0441\u0435 \u043d\u0430 150 \u0441\u0442\u0440\u043e\u043a.<\/em><\/p>\n

\u0421\u0435\u0433\u043e\u0434\u043d\u044f \u2014 \u0432\u0442\u043e\u0440\u0430\u044f \u0447\u0430\u0441\u0442\u044c. \u0422\u0443\u0442 \u043d\u0435\u0442 \u0441\u0442\u0440\u0430\u0448\u043d\u044b\u0445 race conditions, \u043d\u043e \u0435\u0441\u0442\u044c \u0442\u043e, \u0447\u0442\u043e \u0432\u044b\u0434\u0430\u0451\u0442 \u0443\u0440\u043e\u0432\u0435\u043d\u044c \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0430 \u043d\u0430 \u0441\u043e\u0431\u0435\u0441\u0435: <\/em>\u043e\u0442\u043d\u043e\u0448\u0435\u043d\u0438\u0435 \u043a \u043e\u0448\u0438\u0431\u043a\u0430\u043c, \u0432\u0430\u043b\u0438\u0434\u0430\u0446\u0438\u044f, API design, graceful shutdown, observability<\/em><\/strong>. \u042d\u0442\u0438 \u0431\u0430\u0433\u0438 \u043d\u0435 \u0443\u043f\u0430\u0434\u0443\u0442 \u201c\u0432\u0434\u0440\u0443\u0433\u201d \u0432 \u043f\u0440\u043e\u0434\u0430\u043a\u0448\u0435\u043d\u0435 \u2014 \u043e\u043d\u0438 \u0431\u0443\u0434\u0443\u0442 \u0442\u0438\u0445\u043e \u043f\u0438\u043b\u0438\u0442\u044c \u0432\u0430\u043c \u043a\u043e\u0441\u0442\u044b\u043b\u044c \u0437\u0430 \u043a\u043e\u0441\u0442\u044b\u043b\u0451\u043c, \u043f\u043e\u043a\u0430 \u043a\u0442\u043e-\u0442\u043e \u043d\u0435 \u0441\u044f\u0434\u0435\u0442 \u043f\u0435\u0440\u0435\u043f\u0438\u0441\u044b\u0432\u0430\u0442\u044c. \u0410\u043a\u0442\u0443\u0430\u043b\u044c\u043d\u043e \u0434\u043b\u044f Go 1.26.<\/em><\/p>\n

\u041d\u0430\u043f\u043e\u043c\u043d\u044e \u0438\u0442\u043e\u0433 \u043f\u0435\u0440\u0432\u043e\u0439 \u0447\u0430\u0441\u0442\u0438: \u0438\u0437 8 \u0431\u0430\u0433\u043e\u0432 \u043f\u0440\u043e concurrency \u043d\u0430 \u0438\u043d\u0442\u0435\u0440\u0432\u044c\u044e \u043d\u0430\u0448\u0451\u043b 7, \u043f\u0440\u043e\u043f\u0443\u0441\u0442\u0438\u043b \u0442\u043e\u043b\u044c\u043a\u043e TOCTOU race. \u0412 \u044d\u0442\u043e\u0439 \u0447\u0430\u0441\u0442\u0438 \u0438\u0437 13 \u0431\u0430\u0433\u043e\u0432 \u043f\u0440\u043e\u043f\u0443\u0441\u0442\u0438\u043b <\/em>\u0434\u0432\u0430<\/em><\/strong>: <\/em>package applike<\/em><\/code> \u0441 <\/em>func main()<\/em><\/code> (\u0442\u043e, \u0447\u0442\u043e \u043a\u043e\u0434 \u043d\u0435 \u043a\u043e\u043c\u043f\u0438\u043b\u0438\u0440\u0443\u0435\u0442\u0441\u044f \u2014 \u0431\u0430\u043d\u0430\u043b\u044c\u043d\u043e \u043d\u0435 \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0435\u043b \u043d\u0430 \u043e\u0431\u044a\u044f\u0432\u043b\u0435\u043d\u0438\u0435 \u043f\u0430\u043a\u0435\u0442\u0430) \u0438 \u043e\u0442\u0441\u0443\u0442\u0441\u0442\u0432\u0438\u0435 <\/em>slog<\/em><\/code> (\u043f\u0440\u043e\u0441\u0442\u043e \u043d\u0435 \u0437\u0430\u0446\u0435\u043f\u0438\u043b\u0441\u044f \u0437\u0430 <\/em>log.Println<\/em><\/code>, \u0430 \u0437\u0440\u044f). \u041e\u0441\u0442\u0430\u043b\u044c\u043d\u044b\u0435 11 \u2014 \u043f\u043e\u0439\u043c\u0430\u043b. \u0420\u0430\u0441\u0441\u043a\u0430\u0436\u0443, \u043a\u0430\u043a\u0438\u043c\u0438 \u043f\u0430\u0442\u0442\u0435\u0440\u043d\u0430\u043c\u0438 \u0432 \u0447\u0442\u0435\u043d\u0438\u0438 \u043a\u043e\u0434\u0430 \u044f \u0438\u0445 \u0432\u044b\u043b\u0430\u0432\u043b\u0438\u0432\u0430\u043b.<\/em><\/p>\n

\u041d\u0430\u043f\u043e\u043c\u043d\u044e, \u0447\u0442\u043e \u0437\u0430 \u043a\u043e\u0434<\/h3>\n

\u041c\u0438\u043a\u0440\u043e\u0441\u0435\u0440\u0432\u0438\u0441 \u0442\u0440\u0435\u043a\u0438\u043d\u0433\u0430 \u0440\u0435\u043a\u043b\u0430\u043c\u043d\u044b\u0445 \u043a\u043b\u0438\u043a\u043e\u0432 \u043d\u0430 Gin \u0441 in-memory storage. \u041f\u0440\u0438\u043d\u0438\u043c\u0430\u0435\u0442 POST \u0441 (user_uuid, campaign_uuid, country)<\/code>, \u0433\u0435\u043d\u0435\u0440\u0438\u0440\u0443\u0435\u0442 tracking link, \u0434\u0435\u0434\u0443\u043f\u043b\u0438\u0446\u0438\u0440\u0443\u0435\u0442 \u0437\u0430 5 \u0441\u0435\u043a\u0443\u043d\u0434, \u043f\u0430\u0431\u043b\u0438\u0448\u0438\u0442 \u0432 \u043e\u0447\u0435\u0440\u0435\u0434\u044c.<\/p>\n

\u0412 \u043f\u0435\u0440\u0432\u043e\u0439 \u0447\u0430\u0441\u0442\u0438 \u043c\u044b \u043f\u043e\u0447\u0438\u043d\u0438\u043b\u0438 mutex\u2019\u044b, \u0443\u0442\u0435\u0447\u043a\u0438 \u0438 \u043a\u0430\u043d\u0430\u043b\u044b. \u0422\u0435\u043f\u0435\u0440\u044c \u0441\u043c\u043e\u0442\u0440\u0438\u043c \u043d\u0430 \u0432\u0441\u0451 \u043e\u0441\u0442\u0430\u043b\u044c\u043d\u043e\u0435.<\/p>\n

10. \u201c\u0410 return values \u0443 \u043d\u0430\u0441 \u043a\u0430\u043a, \u043f\u043e \u0444\u044d\u043d-\u0448\u0443\u044e?\u201d<\/h3>\n

\u0421\u043c\u043e\u0442\u0440\u044e \u0441\u0438\u0433\u043d\u0430\u0442\u0443\u0440\u0443 GetRecent<\/code>:<\/p>\n

func (r *InMemoryClickRepository) GetRecent(    userUUID, campaignUUID string,     maxAge time.Duration,) (error, *CampaignClick, bool) {    \/\/ ...}<\/code>
<\/a><\/div><\/pre>\n
\u041d\u0430 \u0441\u0438\u0433\u043d\u0430\u0442\u0443\u0440\u0435 \u0433\u043b\u0430\u0437 \u0441\u0430\u043c \u0437\u0430\u0446\u0435\u043f\u0438\u043b\u0441\u044f \u2014 \u043e\u043d\u0430 \u0447\u0438\u0442\u0430\u0435\u0442\u0441\u044f \u043a\u0440\u0438\u0432\u043e, \u043a\u0430\u043a \u0438\u043d\u043e\u0441\u0442\u0440\u0430\u043d\u043d\u044b\u0439 \u0430\u043a\u0446\u0435\u043d\u0442 \u0432 Go-\u043a\u043e\u0434\u0435. \u041e\u0437\u0432\u0443\u0447\u0438\u0432\u0430\u044e: \u201c\u0412 Go \u043a\u043e\u043d\u0432\u0435\u043d\u0446\u0438\u044f \u2014 error<\/code> \u0432\u0441\u0435\u0433\u0434\u0430 \u043f\u043e\u0441\u043b\u0435\u0434\u043d\u0438\u0439. \u0422\u0443\u0442 \u043e\u043d \u043f\u0435\u0440\u0432\u044b\u0439, \u0438 \u043f\u0440\u0438 \u044d\u0442\u043e\u043c \u0444\u0443\u043d\u043a\u0446\u0438\u044f \u0432\u043e\u043e\u0431\u0449\u0435 \u043d\u0438\u043a\u043e\u0433\u0434\u0430 \u0435\u0433\u043e \u043d\u0435 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 (\u0432\u0435\u0437\u0434\u0435 return nil, ...<\/code>). \u041b\u0438\u0448\u043d\u0438\u0439 \u0441\u043b\u043e\u0442 \u0432 \u0441\u0438\u0433\u043d\u0430\u0442\u0443\u0440\u0435 \u2014 \u0432\u044b\u043a\u0438\u043d\u0443\u0442\u044c\u201d.<\/p>\n
\u0412 Go \u044d\u0442\u043e \u043d\u0435\u0433\u043b\u0430\u0441\u043d\u0430\u044f \u043a\u043e\u043d\u0432\u0435\u043d\u0446\u0438\u044f: error \u0432\u0441\u0435\u0433\u0434\u0430 \u043f\u043e\u0441\u043b\u0435\u0434\u043d\u0438\u0439<\/strong>. \u042d\u0442\u043e \u043d\u0435 \u043f\u0440\u043e\u0441\u0442\u043e \u0441\u0442\u0438\u043b\u0435\u0432\u043e\u0435 \u2014 \u044d\u0442\u043e \u0432\u043b\u0438\u044f\u0435\u0442 \u043d\u0430 \u0447\u0438\u0442\u0430\u0435\u043c\u043e\u0441\u0442\u044c, \u043d\u0430 \u0440\u0430\u0431\u043e\u0442\u0443 \u0441 linter\u2019\u0430\u043c\u0438 (errcheck<\/code> \u043e\u0436\u0438\u0434\u0430\u0435\u0442 error \u0432 \u043a\u043e\u043d\u0446\u0435), \u043d\u0430 \u0442\u043e, \u043a\u0430\u043a \u043b\u044e\u0434\u0438 \u0432 \u043a\u043e\u043c\u0430\u043d\u0434\u0435 \u0431\u0443\u0434\u0443\u0442 \u044d\u0442\u043e \u0432\u044b\u0437\u044b\u0432\u0430\u0442\u044c. \u041a\u0430\u0436\u0434\u044b\u0439 Go-\u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a \u043d\u0430 \u0430\u0432\u0442\u043e\u043c\u0430\u0442\u0435 \u043f\u0438\u0448\u0435\u0442:<\/p>\n
val, err := someFunc()<\/code>
<\/a><\/div><\/pre>\n
\u0410 \u0442\u0443\u0442 \u043f\u043e\u043b\u0443\u0447\u0430\u0435\u0442\u0441\u044f:<\/p>\n
err, click, ok := repo.GetRecent(...)<\/code>
<\/a><\/div><\/pre>\n
\u0418 \u0442\u044b \u043a\u0430\u0436\u0434\u044b\u0439 \u0440\u0430\u0437 \u0441\u043f\u043e\u0442\u044b\u043a\u0430\u0435\u0448\u044c\u0441\u044f. \u041f\u043b\u044e\u0441 \u2014 \u0437\u0430\u0447\u0435\u043c \u0442\u0443\u0442 \u0432\u043e\u043e\u0431\u0449\u0435 error<\/code>? \u0424\u0443\u043d\u043a\u0446\u0438\u044f \u043d\u0438\u043a\u043e\u0433\u0434\u0430 \u0435\u0433\u043e \u043d\u0435 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 (\u0432\u0435\u0437\u0434\u0435 return nil, ...<\/code>). \u041b\u0438\u0448\u043d\u0438\u0439 \u0441\u043b\u043e\u0442 \u0432 \u0441\u0438\u0433\u043d\u0430\u0442\u0443\u0440\u0435, \u043a\u043e\u0442\u043e\u0440\u044b\u0439 \u043f\u0443\u0442\u0430\u0435\u0442.<\/p>\n
\u041f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e:<\/p>\n
func (r *InMemoryClickRepository) GetRecent(    userUUID, campaignUUID string,    maxAge time.Duration,) (*CampaignClick, bool) {    \/\/ \u0435\u0441\u043b\u0438 error \u0440\u0435\u0430\u043b\u044c\u043d\u043e \u043d\u0435 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442\u0441\u044f \u2014 \u0443\u0431\u0438\u0440\u0430\u0435\u043c \u0435\u0433\u043e}<\/code>
<\/a><\/div><\/pre>\n
\u041c\u043e\u0440\u0430\u043b\u044c:<\/strong> \u041a\u043e\u043d\u0432\u0435\u043d\u0446\u0438\u0438 \u044f\u0437\u044b\u043a\u0430 \u0441\u0443\u0449\u0435\u0441\u0442\u0432\u0443\u044e\u0442 \u043d\u0435 \u043f\u0440\u043e\u0441\u0442\u043e \u0442\u0430\u043a. error<\/code> \u043f\u043e\u0441\u043b\u0435\u0434\u043d\u0438\u0439, context.Context<\/code> \u043f\u0435\u0440\u0432\u044b\u0439, ok bool<\/code> \u0434\u043b\u044f \u0434\u0432\u043e\u0439\u043d\u043e\u0433\u043e \u0432\u043e\u0437\u0432\u0440\u0430\u0442\u0430 \u0441 map\/cast. \u041d\u0430\u0440\u0443\u0448\u0430\u0435\u0448\u044c \u2014 \u0442\u0432\u043e\u0439 \u043a\u043e\u0434 \u0432\u044b\u0433\u043b\u044f\u0434\u0438\u0442 \u043a\u0430\u043a \u043d\u0430\u043f\u0438\u0441\u0430\u043d\u043d\u044b\u0439 \u0447\u0435\u043b\u043e\u0432\u0435\u043a\u043e\u043c, \u043a\u043e\u0442\u043e\u0440\u044b\u0439 Go \u043d\u0435 \u0437\u043d\u0430\u0435\u0442.<\/p>\n
11. Save() \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 error \u2014 \u0430 \u043c\u044b \u0435\u0433\u043e \u0432\u044b\u0431\u0440\u0430\u0441\u044b\u0432\u0430\u0435\u043c<\/h3>\n
\u0421\u043c\u043e\u0442\u0440\u044e \u043d\u0430 handler:<\/p>\n
c.repo.Save(click)go c.publisher.Publish(click)ctx.JSON(http.StatusOK, ClickResponse{    ClickUUID:    click.ClickUUID,    TrackingLink: click.TrackingLink,})<\/code>
<\/a><\/div><\/pre>\n
Save<\/code> \u043e\u0431\u044a\u044f\u0432\u043b\u0435\u043d \u043a\u0430\u043a func Save(c *CampaignClick) error<\/code>. \u0422\u043e \u0435\u0441\u0442\u044c \u043f\u043e \u043a\u043e\u043d\u0442\u0440\u0430\u043a\u0442\u0443<\/strong> \u043c\u043e\u0436\u0435\u0442 \u0432\u0435\u0440\u043d\u0443\u0442\u044c \u043e\u0448\u0438\u0431\u043a\u0443. \u0410 \u043c\u044b \u0435\u0451 \u043f\u0440\u043e\u0441\u0442\u043e \u0438\u0433\u043d\u043e\u0440\u0438\u0440\u0443\u0435\u043c \u0438 \u043e\u0442\u0432\u0435\u0447\u0430\u0435\u043c \u043a\u043b\u0438\u0435\u043d\u0442\u0443 200 OK, \u043a\u0430\u043a \u0431\u0443\u0434\u0442\u043e \u0432\u0441\u0451 \u0441\u043e\u0445\u0440\u0430\u043d\u0438\u043b\u043e\u0441\u044c.<\/p>\n
\u0421\u0435\u0433\u043e\u0434\u043d\u044f in-memory \u2014 \u0434\u0435\u0439\u0441\u0442\u0432\u0438\u0442\u0435\u043b\u044c\u043d\u043e \u0432\u0441\u0435\u0433\u0434\u0430 nil<\/code>. \u041d\u043e \u0437\u0430\u0432\u0442\u0440\u0430 \u043a\u0442\u043e-\u0442\u043e \u0437\u0430\u043c\u0435\u043d\u0438\u0442 \u043d\u0430 PostgreSQL\/Redis\/Kafka, Save<\/code> \u043d\u0430\u0447\u043d\u0451\u0442 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0442\u044c \u0440\u0435\u0430\u043b\u044c\u043d\u044b\u0435 \u043e\u0448\u0438\u0431\u043a\u0438, \u0430 handler \u043a\u0430\u043a \u043d\u0438 \u0432 \u0447\u0451\u043c \u043d\u0435 \u0431\u044b\u0432\u0430\u043b\u043e \u0440\u0430\u043f\u043e\u0440\u0442\u0443\u0435\u0442 \u201c\u0443\u0441\u043f\u0435\u0445\u201d. \u041a\u043b\u0438\u0435\u043d\u0442 \u043f\u043e\u043b\u0443\u0447\u0430\u0435\u0442 clickUUID, \u0438\u0434\u0451\u0442 \u043f\u043e tracking link, \u043d\u043e \u043d\u0430 \u0441\u0442\u043e\u0440\u043e\u043d\u0435 \u0430\u043d\u0430\u043b\u0438\u0442\u0438\u043a\u0438 \u044d\u0442\u043e\u0433\u043e \u043a\u043b\u0438\u043a\u0430 \u043d\u0435\u0442. Hello, broken funnel.<\/p>\n
\u0412 Go 1.26 \u0437\u0430\u0432\u0435\u0437\u043b\u0438 errors.AsType[T]<\/strong><\/code> \u2014 \u0442\u0438\u043f\u043e\u0431\u0435\u0437\u043e\u043f\u0430\u0441\u043d\u044b\u0439 generic-\u0432\u0430\u0440\u0438\u0430\u043d\u0442 errors.As<\/code><\/a>. \u0423\u0434\u043e\u0431\u043d\u043e \u0434\u043b\u044f \u0447\u0438\u0441\u0442\u043e\u0439 \u043e\u0431\u0440\u0430\u0431\u043e\u0442\u043a\u0438 \u0434\u043e\u043c\u0435\u043d\u043d\u044b\u0445 \u043e\u0448\u0438\u0431\u043e\u043a:<\/p>\n
if err := c.repo.Save(ctx.Request.Context(), click); err != nil {    if dbErr, ok := errors.AsType[*DBError](err); ok && dbErr.Retriable {        \/\/ retry \u0438\u043b\u0438 fallback    }    log.Printf(\"save click failed: %v\", err)    ctx.JSON(http.StatusInternalServerError, gin.H{\"error\": \"internal\"})    return}<\/code>
<\/a><\/div><\/pre>\n
\u041c\u043e\u0440\u0430\u043b\u044c:<\/strong> \u0415\u0441\u043b\u0438 \u0444\u0443\u043d\u043a\u0446\u0438\u044f \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 error<\/code> \u2014 \u043e\u0431\u0440\u0430\u0431\u043e\u0442\u0430\u0439 \u0435\u0433\u043e. \u0412\u0441\u0435\u0433\u0434\u0430. \u0414\u0430\u0436\u0435 \u0435\u0441\u043b\u0438 \u201c\u0441\u0435\u0439\u0447\u0430\u0441 \u043e\u043d \u0432\u0441\u0435\u0433\u0434\u0430 nil\u201d. \u0421\u0438\u0433\u043d\u0430\u0442\u0443\u0440\u0430 \u2014 \u044d\u0442\u043e \u043a\u043e\u043d\u0442\u0440\u0430\u043a\u0442 \u043d\u0430 \u0431\u0443\u0434\u0443\u0449\u0435\u0435. errcheck<\/code> linter \u0432 CI \u0437\u0430\u043a\u0440\u043e\u0435\u0442 90% \u0442\u0430\u043a\u0438\u0445 \u0434\u044b\u0440 \u0430\u0432\u0442\u043e\u043c\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438.<\/p>\n
12. \u0412\u0430\u043b\u0438\u0434\u0430\u0446\u0438\u044f? \u041d\u0435, \u043d\u0435 \u0441\u043b\u044b\u0448\u0430\u043b\u0438<\/h3>\n
\u0421\u043c\u043e\u0442\u0440\u044e handler:<\/p>\n
func (c *ClickController) Post(ctx *gin.Context) {    var req struct {        UserUUID     string `json:\"user_uuid\"`        CampaignUUID string `json:\"campaign_uuid\"`        Country      string `json:\"country\"`    }    _ = ctx.ShouldBindJSON(&req)    \/\/ \u0434\u0430\u043b\u044c\u0448\u0435 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u0435\u043c req...}<\/code>
<\/a><\/div><\/pre>\n
\u0417\u0430\u043c\u0435\u0442\u0438\u043b\u0438?  = ctx.ShouldBindJSON(&req)<\/em><\/code>. \u041e\u0448\u0438\u0431\u043a\u0430 \u044f\u0432\u043d\u043e \u043f\u0440\u043e\u0438\u0433\u043d\u043e\u0440\u0438\u0440\u043e\u0432\u0430\u043d\u0430 \u0447\u0435\u0440\u0435\u0437 «_ «<\/em>. \u0422\u043e \u0435\u0441\u0442\u044c \u0435\u0441\u043b\u0438 \u043f\u0440\u0438\u0434\u0451\u0442 \u043d\u0435\u0432\u0430\u043b\u0438\u0434\u043d\u044b\u0439 JSON, \u043c\u044b \u043f\u043e\u043b\u0443\u0447\u0438\u043c \u043f\u0443\u0441\u0442\u0443\u044e \u0441\u0442\u0440\u0443\u043a\u0442\u0443\u0440\u0443<\/strong>, \u0430 \u043f\u043e\u0442\u043e\u043c \u0441\u043e\u0445\u0440\u0430\u043d\u0438\u043c \u0432 \u0440\u0435\u043f\u043e\u0437\u0438\u0442\u043e\u0440\u0438\u0439 \u0437\u0430\u043f\u0438\u0441\u044c \u0441 \u043f\u0443\u0441\u0442\u044b\u043c\u0438 UUID\u2019\u0430\u043c\u0438. \u0417\u0430\u0442\u0435\u043c \u0441\u0433\u0435\u043d\u0435\u0440\u0438\u0440\u0443\u0435\u043c tracking link https:\/\/tracker.example\/click?campaign=&user=<\/code><\/a> \u0438 \u043e\u0442\u043f\u0440\u0430\u0432\u0438\u043c \u043a\u043b\u0438\u0435\u043d\u0442\u0443. Hello broken analytics.<\/p>\n
\u0418 \u0432\u0430\u043b\u0438\u0434\u0430\u0446\u0438\u0438 \u043d\u0435\u0442 \u0432\u043e\u043e\u0431\u0449\u0435 \u043d\u0438\u043a\u0430\u043a\u043e\u0439. UUID \u043c\u043e\u0433 \u0431\u044b\u0442\u044c \u0441\u0442\u0440\u043e\u043a\u043e\u0439 \"drop table users\"<\/code>, \u0441\u0442\u0440\u0430\u043d\u0430 \u2014 \u043f\u0443\u0441\u0442\u043e\u0439, \u0438\u043b\u0438 \"zz\"<\/code>, \u0438\u043b\u0438 \"\ud83c\uddf7\ud83c\uddfa\"<\/code>. \u0412\u0441\u0451 \u044d\u0442\u043e \u0443\u043b\u0435\u0442\u0438\u0442 \u0432 \u0411\u0414 (\u043a\u043e\u0433\u0434\u0430 in-memory \u0437\u0430\u043c\u0435\u043d\u044f\u0442 \u043d\u0430 \u043f\u043e\u0441\u0442\u0433\u0440\u0435\u0441) \u0438 \u0432 metrics.<\/p>\n
\u041f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e \u2014 Gin \u0443\u043c\u0435\u0435\u0442 \u0432\u0430\u043b\u0438\u0434\u0438\u0440\u043e\u0432\u0430\u0442\u044c \u0441\u0430\u043c \u0447\u0435\u0440\u0435\u0437 binding<\/code> \u0442\u0435\u0433\u0438:<\/p>\n
func (c *ClickController) Post(ctx *gin.Context) {    var req struct {        UserUUID     string `json:\"user_uuid\"     binding:\"required,uuid\"`        CampaignUUID string `json:\"campaign_uuid\" binding:\"required,uuid\"`        Country      string `json:\"country\"       binding:\"required,len=2,alpha\"`    }    if err := ctx.ShouldBindJSON(&req); err != nil {        ctx.JSON(http.StatusBadRequest, gin.H{\"error\": err.Error()})        return    }    \/\/ \u0434\u0430\u043b\u044c\u0448\u0435 \u043c\u043e\u0436\u043d\u043e \u0434\u043e\u0432\u0435\u0440\u044f\u0442\u044c req}<\/code>
<\/a><\/div><\/pre>\n
\u041c\u043e\u0440\u0430\u043b\u044c:<\/strong> \u041b\u044e\u0431\u043e\u0439 external input \u2014 \u044d\u0442\u043e \u0432\u0440\u0430\u0436\u0434\u0435\u0431\u043d\u0430\u044f \u0441\u0440\u0435\u0434\u0430<\/strong>. \u0412\u0430\u043b\u0438\u0434\u0438\u0440\u0443\u0439 \u043d\u0430 \u0433\u0440\u0430\u043d\u0438\u0446\u0435 \u0441\u0438\u0441\u0442\u0435\u043c\u044b. \u0418 \u041d\u0418\u041a\u041e\u0413\u0414\u0410 \u043d\u0435 \u0438\u0433\u043d\u043e\u0440\u0438\u0440\u0443\u0439 \u043e\u0448\u0438\u0431\u043a\u0438 \u0447\u0435\u0440\u0435\u0437 «_». \u042d<\/em>\u0442\u043e \u0432\u0441\u0435\u0433\u0434\u0430 \u0441\u0440\u0430\u0437\u0443 \u0442\u0440\u0438\u0433\u0433\u0435\u0440 \u043d\u0430 code review: \u043b\u0438\u0431\u043e \u044f\u0432\u043d\u043e \u043e\u0448\u0438\u0431\u043a\u0430 \u043f\u0440\u043e\u0438\u0433\u043d\u043e\u0440\u0438\u0440\u043e\u0432\u0430\u043d\u0430 \u0438 \u044d\u0442\u043e \u041e\u041a (\u0438 \u0442\u043e\u0433\u0434\u0430 \u043d\u0443\u0436\u0435\u043d \u043a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439 \u2014 \u043f\u043e\u0447\u0435\u043c\u0443), \u043b\u0438\u0431\u043e \u043e\u0448\u0438\u0431\u043a\u0430 \u043f\u0440\u043e\u0438\u0433\u043d\u043e\u0440\u0438\u0440\u043e\u0432\u0430\u043d\u0430 \u0438 \u044d\u0442\u043e \u041d\u0415<\/strong> \u041e\u041a. \u0422\u0440\u0435\u0442\u044c\u0435\u0433\u043e \u043d\u0435 \u0434\u0430\u043d\u043e.<\/p>\n
13. Country \u0431\u0435\u0437 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 \u043d\u0430 ISO 3166-1<\/h3>\n
\u0414\u0430\u0436\u0435 \u0435\u0441\u043b\u0438 \u0434\u043e\u0431\u0430\u0432\u0438\u0442\u044c binding:\"len=2,alpha\"<\/code>, \u044d\u0442\u043e \u043f\u0440\u043e\u043f\u0443\u0441\u0442\u0438\u0442 \"XQ\"<\/code>, \"ZZ\"<\/code>, \"AA\"<\/code> \u2014 \u0444\u043e\u0440\u043c\u0430\u043b\u044c\u043d\u043e 2 \u0431\u0443\u043a\u0432\u044b, \u043d\u043e \u043d\u0435 \u0441\u0443\u0449\u0435\u0441\u0442\u0432\u0443\u044e\u0449\u0438\u0435 \u043a\u043e\u0434\u044b \u0441\u0442\u0440\u0430\u043d. \u0412 \u0430\u043d\u0430\u043b\u0438\u0442\u0438\u043a\u0435 \u043f\u043e\u0442\u043e\u043c \u044d\u0442\u0438 \u201c\u0441\u0442\u0440\u0430\u043d\u044b\u201d \u0441\u0438\u0434\u044f\u0442 \u043e\u0442\u0434\u0435\u043b\u044c\u043d\u043e\u0439 \u0441\u0442\u0440\u043e\u043a\u043e\u0439 \u0438 \u043f\u043e\u0440\u0442\u044f\u0442 \u0433\u0440\u0430\u0444\u0438\u043a\u0438.<\/p>\n
\u041f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e \u2014 \u043a\u0430\u0441\u0442\u043e\u043c\u043d\u044b\u0439 validator \u0441 whitelist\u2019\u043e\u043c:<\/p>\n
import \"github.com\/biter777\/countries\"func validateCountry(fl validator.FieldLevel) bool {    code := fl.Field().String()    return countries.ByName(code).IsValid()}\/\/ \u043f\u0440\u0438 \u0438\u043d\u0438\u0446\u0438\u0430\u043b\u0438\u0437\u0430\u0446\u0438\u0438:if v, ok := binding.Validator.Engine().(*validator.Validate); ok {    v.RegisterValidation(\"iso3166\", validateCountry)}\/\/ \u0432 struct'\u0435:Country string `binding:\"required,iso3166\"`<\/code>
<\/a><\/div><\/pre>\n
\u041b\u0438\u0431\u043e \u0445\u0440\u0430\u043d\u0438\u0442\u044c enum\u2019\u043e\u043c \u0438 \u043f\u0430\u0440\u0441\u0438\u0442\u044c:<\/p>\n
type CountryCode stringfunc (c *CountryCode) UnmarshalJSON(data []byte) error {    var s string    if err := json.Unmarshal(data, &s); err != nil {        return err    }    if !isValidISO3166(s) {        return fmt.Errorf(\"invalid country code: %q\", s)    }    *c = CountryCode(s)    return nil}<\/code>
<\/a><\/div><\/pre>\n
\u041c\u043e\u0440\u0430\u043b\u044c:<\/strong> \u201c\u042d\u0442\u043e \u0436\u0435 \u043f\u0440\u043e\u0441\u0442\u043e \u0441\u0442\u0440\u043e\u043a\u0430\u201d \u2014 \u0433\u043b\u0430\u0432\u043d\u044b\u0439 \u0438\u0441\u0442\u043e\u0447\u043d\u0438\u043a \u043c\u0443\u0441\u043e\u0440\u0430 \u0432 \u0442\u0430\u0431\u043b\u0438\u0446\u0430\u0445. \u0415\u0441\u043b\u0438 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 \u0438\u0437 \u043a\u043e\u043d\u0435\u0447\u043d\u043e\u0433\u043e \u043d\u0430\u0431\u043e\u0440\u0430 \u2014 \u0437\u0430\u0432\u0435\u0434\u0438 enum \u0438\u043b\u0438 \u0432\u0430\u043b\u0438\u0434\u0430\u0442\u043e\u0440. Domain types > primitive types.<\/p>\n
14. Tracking link \u0441\u043e\u0431\u0440\u0430\u043d \u0441\u0442\u0440\u043e\u043a\u043e\u0439 \u2014 \u0430 \u0442\u0430\u043c \u0431\u0430\u0433<\/h3>\n
\u0412\u043d\u0438\u043c\u0430\u0442\u0435\u043b\u044c\u043d\u043e \u0441\u043c\u043e\u0442\u0440\u044e \u043d\u0430:<\/p>\n
func generateTrackingLink(campaignUUID, userUUID string) string {    return \"https:\/\/tracker.example\/click?campaign=\" + campaignUUID + \"?user=\" + userUUID}<\/code>
<\/a><\/div><\/pre>\n
\u0412\u0438\u0434\u0438\u0442\u0435? \u0414\u0432\u0430 \u0437\u043d\u0430\u043a\u0430 <\/strong>?<\/strong><\/code> \u0432 URL.<\/strong> \u0414\u043e\u043b\u0436\u0435\u043d \u0431\u044b\u0442\u044c ?<\/code> \u0434\u043b\u044f \u043f\u0435\u0440\u0432\u043e\u0433\u043e \u043f\u0430\u0440\u0430\u043c\u0435\u0442\u0440\u0430 \u0438 &<\/code> \u0434\u043b\u044f \u043e\u0441\u0442\u0430\u043b\u044c\u043d\u044b\u0445. \u0422\u043e \u0435\u0441\u0442\u044c \u0444\u0430\u043a\u0442\u0438\u0447\u0435\u0441\u043a\u0438 \u0432 ?campaign=...?user=...<\/code> \u043f\u0430\u0440\u0430\u043c\u0435\u0442\u0440 campaign<\/code> \u0431\u0443\u0434\u0435\u0442 \u0441\u043e\u0434\u0435\u0440\u0436\u0430\u0442\u044c \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 <uuid>?user=<uuid><\/code>, \u0430 \u043f\u0430\u0440\u0430\u043c\u0435\u0442\u0440\u0430 user<\/code> \u0432\u043e\u043e\u0431\u0449\u0435 \u043d\u0435 \u0431\u0443\u0434\u0435\u0442 \u043d\u0430 \u0441\u0442\u043e\u0440\u043e\u043d\u0435 tracker\u2019\u0430.<\/p>\n
\u042d\u0442\u043e \u0434\u0430\u0436\u0435 \u043d\u0435 \u0431\u0430\u0433 concurrency \u0438\u043b\u0438 \u043f\u0440\u043e\u0438\u0437\u0432\u043e\u0434\u0438\u0442\u0435\u043b\u044c\u043d\u043e\u0441\u0442\u0438. \u042d\u0442\u043e \u0431\u0438\u0437\u043d\u0435\u0441-\u0431\u0430\u0433<\/strong>: \u0442\u0440\u0435\u043a\u0435\u0440 \u043d\u0435 \u0441\u043c\u043e\u0436\u0435\u0442 \u0430\u0442\u0440\u0438\u0431\u0443\u0442\u0438\u0440\u043e\u0432\u0430\u0442\u044c \u043a\u043b\u0438\u043a \u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u0435\u043b\u044e. \u0414\u0435\u043d\u044c\u0433\u0438 \u0440\u0435\u043a\u043b\u0430\u043c\u043e\u0434\u0430\u0442\u0435\u043b\u0435\u0439 \u0443\u043b\u0435\u0442\u0430\u044e\u0442 \u0432 \u043d\u0438\u043a\u0443\u0434\u0430, \u043f\u043e\u0442\u043e\u043c\u0443 \u0447\u0442\u043e \u0430\u043d\u0430\u043b\u0438\u0442\u0438\u043a\u0430 \u0431\u0438\u0442\u0430\u044f.<\/p>\n
\u041c\u043e\u0440\u0430\u043b\u044c:<\/strong> \u041d\u0435 \u043a\u043b\u0435\u0439 URL\u2019\u044b \u0438 SQL-\u0437\u0430\u043f\u0440\u043e\u0441\u044b \u0441\u0442\u0440\u043e\u043a\u0430\u043c\u0438. \u0414\u043b\u044f URL \u2014 net\/url<\/code>, \u0434\u043b\u044f SQL \u2014 \u043f\u0430\u0440\u0430\u043c\u0435\u0442\u0440\u0438\u0437\u043e\u0432\u0430\u043d\u043d\u044b\u0435 \u0437\u0430\u043f\u0440\u043e\u0441\u044b. \u042d\u0442\u043e \u043f\u0440\u0430\u0432\u0438\u043b\u043e 1990-\u0445, \u043a\u043e\u0442\u043e\u0440\u043e\u0435 \u043f\u043e\u0447\u0435\u043c\u0443-\u0442\u043e \u0432\u0441\u0451 \u0435\u0449\u0451 \u043d\u0430\u0440\u0443\u0448\u0430\u044e\u0442 \u0432 2026-\u043c.<\/p>\n
15. URL \u0431\u0435\u0437 escaping \u2014 open \u0434\u043b\u044f injection<\/h3>\n
\u0414\u043e\u043f\u0443\u0441\u0442\u0438\u043c, \u043c\u044b \u043f\u043e\u0447\u0438\u043d\u0438\u043b\u0438 \u043f\u0440\u043e\u0448\u043b\u044b\u0439 \u0431\u0430\u0433 \u0438 \u0441\u043e\u0431\u0438\u0440\u0430\u0435\u043c URL \u0440\u0443\u043a\u0430\u043c\u0438: ?campaign=<\/code> + &user=<\/code>. \u0412\u0441\u0451 \u0435\u0449\u0451 \u043f\u043b\u043e\u0445\u043e: \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f \u043d\u0435 \u044d\u043a\u0440\u0430\u043d\u0438\u0440\u0443\u044e\u0442\u0441\u044f<\/strong>. \u0415\u0441\u043b\u0438 \u0432 userUUID<\/code> \u043f\u0440\u0438\u043b\u0435\u0442\u0438\u0442 \u0441\u0442\u0440\u043e\u043a\u0430 \u0441 &<\/code> \u0438\u043b\u0438 =<\/code> (\u0447\u0435\u0440\u0435\u0437 \u0442\u043e\u0442 \u0441\u0430\u043c\u044b\u0439 \u043e\u0442\u0441\u0443\u0442\u0441\u0442\u0432\u0443\u044e\u0449\u0438\u0439 \u0432\u0430\u043b\u0438\u0434\u0430\u0442\u043e\u0440 \u0438\u0437 \u0431\u0430\u0433\u0430 #12), \u043c\u044b \u0441\u043b\u043e\u043c\u0430\u0435\u043c \u0440\u0430\u0437\u0431\u043e\u0440 query string \u043d\u0430 \u0441\u0442\u043e\u0440\u043e\u043d\u0435 tracker\u2019\u0430:<\/p>\n
?campaign=abc&user=evil&fake=injected<\/code>
<\/a><\/div><\/pre>\n
\u041f\u0430\u0440\u0430\u043c\u0435\u0442\u0440 user<\/code> \u0441\u0442\u0430\u043b evil<\/code>, \u0430 \u0432 URL \u043f\u043e\u044f\u0432\u0438\u043b\u0441\u044f \u043b\u0435\u0432\u044b\u0439 fake<\/code>. \u0415\u0441\u043b\u0438 tracker \u044d\u0442\u043e\u043c\u0443 \u0434\u043e\u0432\u0435\u0440\u044f\u0435\u0442 \u2014 \u0443 \u043d\u0430\u0441 injection.<\/p>\n
\u041f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e \u2014 net\/url<\/code>:<\/p>\n
import \"net\/url\"func generateTrackingLink(campaignUUID, userUUID string) string {    u, _ := url.Parse(\"https:\/\/tracker.example\/click\")    q := u.Query()    q.Set(\"campaign\", campaignUUID)    q.Set(\"user\", userUUID)    u.RawQuery = q.Encode()  \/\/ \u0430\u0432\u0442\u043e\u043c\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0439 escape    return u.String()}<\/code>
<\/a><\/div><\/pre>\n
url.Values.Encode()<\/code> \u0441\u0434\u0435\u043b\u0430\u0435\u0442 URL-escaping \u0430\u0432\u0442\u043e\u043c\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438. \u0417\u0430\u043e\u0434\u043d\u043e \u043f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e \u0440\u0430\u0441\u0441\u0442\u0430\u0432\u0438\u0442 ?<\/code> \u0438 &<\/code>.<\/p>\n
\u041c\u043e\u0440\u0430\u043b\u044c:<\/strong> \u041b\u044e\u0431\u044b\u0435 user-controlled \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f, \u043f\u043e\u043f\u0430\u0434\u0430\u044e\u0449\u0438\u0435 \u0432 URL\/SQL\/HTML \u2014 \u0434\u043e\u043b\u0436\u043d\u044b \u043f\u0440\u043e\u0445\u043e\u0434\u0438\u0442\u044c \u0447\u0435\u0440\u0435\u0437 \u0441\u043e\u043e\u0442\u0432\u0435\u0442\u0441\u0442\u0432\u0443\u044e\u0449\u0438\u0439 escape. net\/url<\/code>, database\/sql<\/code> (placeholders), html\/template<\/code> (\u0430 \u043d\u0435 text\/template<\/code>) \u2014 \u044d\u0442\u043e \u0431\u0430\u0437\u043e\u0432\u0430\u044f \u0433\u0438\u0433\u0438\u0435\u043d\u0430. \u041d\u0435 \u0434\u0435\u043b\u0430\u0439 \u0440\u0443\u043a\u0430\u043c\u0438 \u0442\u043e, \u0447\u0442\u043e \u0441\u0442\u0430\u043d\u0434\u0430\u0440\u0442\u043d\u0430\u044f \u0431\u0438\u0431\u043b\u0438\u043e\u0442\u0435\u043a\u0430 \u0434\u0435\u043b\u0430\u0435\u0442 \u043a\u043e\u0440\u0440\u0435\u043a\u0442\u043d\u043e.<\/p>\n
16. Status 200 \u0434\u043b\u044f \u0441\u043e\u0437\u0434\u0430\u043d\u043d\u043e\u0433\u043e \u0440\u0435\u0441\u0443\u0440\u0441\u0430<\/h3>\n
Handler \u0432\u0441\u0435\u0433\u0434\u0430 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 http.StatusOK<\/code>:<\/p>\n
ctx.JSON(http.StatusOK, ClickResponse{...})<\/code>
<\/a><\/div><\/pre>\n
\u0418 \u0434\u043b\u044f \u043d\u043e\u0432\u043e\u0433\u043e<\/strong> \u043a\u043b\u0438\u043a\u0430, \u0438 \u0434\u043b\u044f \u0434\u0435\u0434\u0443\u043f\u043b\u0438\u0446\u0438\u0440\u043e\u0432\u0430\u043d\u043d\u043e\u0433\u043e<\/strong> \u0432\u043e\u0437\u0432\u0440\u0430\u0442\u0430 \u0438\u0437 \u043a\u0435\u0448\u0430. \u0421\u0435\u043c\u0430\u043d\u0442\u0438\u0447\u0435\u0441\u043a\u0438 \u044d\u0442\u043e \u043d\u0435\u0432\u0435\u0440\u043d\u043e:<\/p>\n