mirror of
				https://github.com/elyby/chrly.git
				synced 2025-05-31 14:11:51 +05:30 
			
		
		
		
	Export profile requests metrics to statsd
This commit is contained in:
		| @@ -5,6 +5,11 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), | |||||||
| and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). | and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). | ||||||
|  |  | ||||||
| ## [Unreleased] - xxxx-xx-xx | ## [Unreleased] - xxxx-xx-xx | ||||||
|  | ### Added | ||||||
|  | - New StatsD metrics: | ||||||
|  |   - Counters: | ||||||
|  |     - `ely.skinsystem.{hostname}.app.profiles.request` | ||||||
|  |  | ||||||
| ### Changed | ### Changed | ||||||
| - Bumped Go version to 1.21. | - Bumped Go version to 1.21. | ||||||
|  |  | ||||||
|   | |||||||
| @@ -133,6 +133,8 @@ func (s *StatsReporter) handleBeforeRequest(req *http.Request) { | |||||||
| 		key = "signed_textures.request" | 		key = "signed_textures.request" | ||||||
| 	} else if strings.HasPrefix(p, "/textures/") { | 	} else if strings.HasPrefix(p, "/textures/") { | ||||||
| 		key = "textures.request" | 		key = "textures.request" | ||||||
|  | 	} else if strings.HasPrefix(p, "/profile/") { | ||||||
|  | 		key = "profiles.request" | ||||||
| 	} else if m == http.MethodPost && p == "/api/skins" { | 	} else if m == http.MethodPost && p == "/api/skins" { | ||||||
| 		key = "api.skins.post.request" | 		key = "api.skins.post.request" | ||||||
| 	} else if m == http.MethodDelete && strings.HasPrefix(p, "/api/skins/") { | 	} else if m == http.MethodDelete && strings.HasPrefix(p, "/api/skins/") { | ||||||
|   | |||||||
| @@ -99,6 +99,14 @@ var statsReporterTestCases = []*StatsReporterTestCase{ | |||||||
| 			{"IncCounter", "signed_textures.request", int64(1)}, | 			{"IncCounter", "signed_textures.request", int64(1)}, | ||||||
| 		}, | 		}, | ||||||
| 	}, | 	}, | ||||||
|  | 	{ | ||||||
|  | 		Events: [][]interface{}{ | ||||||
|  | 			{"skinsystem:before_request", httptest.NewRequest("GET", "http://localhost/profile/username", nil)}, | ||||||
|  | 		}, | ||||||
|  | 		ExpectedCalls: [][]interface{}{ | ||||||
|  | 			{"IncCounter", "profiles.request", int64(1)}, | ||||||
|  | 		}, | ||||||
|  | 	}, | ||||||
| 	{ | 	{ | ||||||
| 		Events: [][]interface{}{ | 		Events: [][]interface{}{ | ||||||
| 			{"skinsystem:before_request", httptest.NewRequest("POST", "http://localhost/api/skins", nil)}, | 			{"skinsystem:before_request", httptest.NewRequest("POST", "http://localhost/api/skins", nil)}, | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user