Skip to content

Commit 2ec0980

Browse files
sonar fix
1 parent b5a09e9 commit 2ec0980

File tree

2 files changed

+7
-8
lines changed

2 files changed

+7
-8
lines changed

src/eligibility_signposting_api/config/config.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ def config() -> dict[str, Any]:
2323
rules_bucket_name = BucketName(os.getenv("RULES_BUCKET_NAME", "test-rules-bucket"))
2424
audit_bucket_name = BucketName(os.getenv("AUDIT_BUCKET_NAME", "test-audit-bucket"))
2525
aws_default_region = AwsRegion(os.getenv("AWS_DEFAULT_REGION", "eu-west-1"))
26-
api_domain_name = ApiDomainName(os.getenv("API_DOMAIN_NAME", "localhost"))
2726
enable_xray_patching = bool(os.getenv("ENABLE_XRAY_PATCHING", "false"))
2827
kinesis_audit_stream_to_s3 = AwsKinesisFirehoseStreamName(
2928
os.getenv("KINESIS_AUDIT_STREAM_TO_S3", "test_kinesis_audit_stream_to_s3")
@@ -41,7 +40,6 @@ def config() -> dict[str, Any]:
4140
"rules_bucket_name": rules_bucket_name,
4241
"audit_bucket_name": audit_bucket_name,
4342
"firehose_endpoint": None,
44-
"api_domain_name": api_domain_name,
4543
"kinesis_audit_stream_to_s3": kinesis_audit_stream_to_s3,
4644
"enable_xray_patching": enable_xray_patching,
4745
"log_level": log_level,
@@ -58,7 +56,6 @@ def config() -> dict[str, Any]:
5856
"rules_bucket_name": rules_bucket_name,
5957
"audit_bucket_name": audit_bucket_name,
6058
"firehose_endpoint": URL(os.getenv("FIREHOSE_ENDPOINT", local_stack_endpoint)),
61-
"api_domain_name": api_domain_name,
6259
"kinesis_audit_stream_to_s3": kinesis_audit_stream_to_s3,
6360
"enable_xray_patching": enable_xray_patching,
6461
"log_level": log_level,

src/eligibility_signposting_api/views/eligibility.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,13 @@
11
import logging
2+
import os
23
import uuid
34
from datetime import UTC, datetime
45
from http import HTTPStatus
5-
from typing import Annotated, Any
6+
from typing import Any
67

78
from flask import Blueprint, make_response, request
89
from flask.typing import ResponseReturnValue
9-
from wireup import Inject, Injected
10+
from wireup import Injected
1011

1112
from eligibility_signposting_api.audit.audit_context import AuditContext
1213
from eligibility_signposting_api.audit.audit_service import AuditService
@@ -35,8 +36,8 @@ def before_request() -> None:
3536

3637

3738
@eligibility_blueprint.get("/_status")
38-
def api_status(api_domain_name: Annotated[str, Inject(param="api_domain_name")]) -> ResponseReturnValue:
39-
return make_response(build_status_payload(api_domain_name), HTTPStatus.OK)
39+
def api_status() -> ResponseReturnValue:
40+
return make_response(build_status_payload(), HTTPStatus.OK)
4041

4142

4243
@eligibility_blueprint.get("/", defaults={"nhs_number": ""})
@@ -179,7 +180,8 @@ def build_suitability_results(condition: Condition) -> list[eligibility_response
179180
]
180181

181182

182-
def build_status_payload(api_domain_name: str) -> dict:
183+
def build_status_payload() -> dict:
184+
api_domain_name = os.getenv("API_DOMAIN_NAME", "localhost")
183185
return {
184186
"status": "pass",
185187
"version": "",

0 commit comments

Comments
 (0)