-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Get Establishment by Trust Repository method #410
Conversation
return NotFound(); | ||
} | ||
|
||
_logger.LogInformation($"Returning Establishments for Trust with specific UKPRN : {commaSeparatedRequestTrust}"); |
Check notice
Code scanning / SonarCloud
Logging should not be vulnerable to injection attacks Low
public async Task<ActionResult<List<EstablishmentDto>>> GetByTrust([FromQuery] string trustUkprn, CancellationToken cancellationToken) | ||
{ | ||
var commaSeparatedRequestTrust = string.Join(",", trustUkprn); | ||
_logger.LogInformation($"Attemping to get establishments by Trust UKPRN : {commaSeparatedRequestTrust}"); |
Check notice
Code scanning / SonarCloud
Logging should not be vulnerable to injection attacks Low
|
||
if (establishments == null) | ||
{ | ||
_logger.LogInformation($"No establishment was found with the requested Trust UKPRN : {commaSeparatedRequestTrust}"); |
Check notice
Code scanning / SonarCloud
Logging should not be vulnerable to injection attacks Low
return NotFound(); | ||
} | ||
|
||
_logger.LogInformation($"Returning trust found by Companies House Number {companiesHouseNumber}"); |
Check notice
Code scanning / SonarCloud
Logging should not be vulnerable to injection attacks Low
[SwaggerResponse(404, "Trust with specified Companies House Number not found.")] | ||
public async Task<ActionResult<TrustDto>> GetTrustByCompaniesHouseNumber(string companiesHouseNumber, CancellationToken cancellationToken) | ||
{ | ||
_logger.LogInformation($"Attempting to get trust by Companies House Number {companiesHouseNumber}"); |
Check notice
Code scanning / SonarCloud
Logging should not be vulnerable to injection attacks Low
|
||
if (trust == null) | ||
{ | ||
_logger.LogInformation($"No trust found for Companies House Number {companiesHouseNumber}"); |
Check notice
Code scanning / SonarCloud
Logging should not be vulnerable to injection attacks Low
Kudos, SonarCloud Quality Gate passed! |
No description provided.