vbatts/registry
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #2680 from manishtomar/mani-fix-mem-leak

Browse source 
fix memory leak introduced in PR #2648
This commit is contained in:
Olivier Gambier 2018-08-24 14:35:12 -07:00 committed by GitHub
commit 3354cf98e3
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
registry/handlers/app.go
View file

@ -706,7 +706,7 @@ func (app *App) dispatcher(dispatch dispatchFunc) http.Handler {
} }
// assign and decorate the authorized repository with an event bridge. // assign and decorate the authorized repository with an event bridge.
context.Repository, context.App.repoRemover = notifications.Listen( context.Repository, context.RepositoryRemover = notifications.Listen(
repository, repository,
context.App.repoRemover, context.App.repoRemover,
app.eventBridge(context, r)) app.eventBridge(context, r))

3
registry/handlers/context.go
View file

@ -25,6 +25,9 @@ type Context struct {
// should be scoped to a single repository. This field may be nil. // should be scoped to a single repository. This field may be nil.
Repository distribution.Repository Repository distribution.Repository
// RepositoryRemover provides method to delete a repository
RepositoryRemover distribution.RepositoryRemover
// Errors is a collection of errors encountered during the request to be // Errors is a collection of errors encountered during the request to be
// returned to the client API. If errors are added to the collection, the // returned to the client API. If errors are added to the collection, the
// handler *must not* start the response via http.ResponseWriter. // handler *must not* start the response via http.ResponseWriter.