Allow anon access and make package finding work OK

bartosz-jarmuz · bartosz-jarmuz · commit eea088e94eec · 2019-04-18T16:50:52.000+02:00
diff --git a/Telimena/Telimena.WebApp/Controllers/Api/V1/ToolkitController.cs b/Telimena/Telimena.WebApp/Controllers/Api/V1/ToolkitController.cs
@@ -58,10 +58,12 @@ public ToolkitController(IToolkitDataUnitOfWork work, IFileSaver fileSaver, IFil
         /// <param name="id"></param>
         /// <returns></returns>
         [Audit]
+        [AllowAnonymous]
         [HttpGet, Route("{id}", Name = Routes.Get)]
         public async Task<IHttpActionResult> Get(Guid id)
         {
-            TelimenaToolkitData toolkitData = await this.work.ToolkitDataRepository.FirstOrDefaultAsync(x => x.PublicId == id).ConfigureAwait(false);
+            TelimenaToolkitData toolkitData = await this.work.ToolkitDataRepository.FirstOrDefaultAsync(x
+                => x.TelimenaPackageInfo != null && x.TelimenaPackageInfo.PublicId == id).ConfigureAwait(false);
             if (toolkitData == null)
             {
                 return this.BadRequest($"Toolkit id [{id}] does not exist");

Original file line number	Diff line number	Diff line change
`@@ -58,10 +58,12 @@ public ToolkitController(IToolkitDataUnitOfWork work, IFileSaver fileSaver, IFil`
`58`	`58`	`/// <param name="id"></param>`
`59`	`59`	`/// <returns></returns>`
`60`	`60`	`[Audit]`
	`61`	`+ [AllowAnonymous]`
`61`	`62`	`[HttpGet, Route("{id}", Name = Routes.Get)]`
`62`	`63`	`public async Task<IHttpActionResult> Get(Guid id)`
`63`	`64`	`{`
`64`		`- TelimenaToolkitData toolkitData = await this.work.ToolkitDataRepository.FirstOrDefaultAsync(x => x.PublicId == id).ConfigureAwait(false);`
	`65`	`+ TelimenaToolkitData toolkitData = await this.work.ToolkitDataRepository.FirstOrDefaultAsync(x`
	`66`	`+ => x.TelimenaPackageInfo != null && x.TelimenaPackageInfo.PublicId == id).ConfigureAwait(false);`
`65`	`67`	`if (toolkitData == null)`
`66`	`68`	`{`
`67`	`69`	`return this.BadRequest($"Toolkit id [{id}] does not exist");`