From 76a84f6e64f39e1e3f3683f7debd3080e5494f88 Mon Sep 17 00:00:00 2001 From: Annie Tran Date: Tue, 26 Mar 2024 09:16:24 -0500 Subject: [PATCH 1/5] [DR-74493] fix pdf validation config in prod (#16067) * use Benefits Intake API key in production; default to DR API key in lower environments as originally designed * rubocop nits --- .../utilities/pdf_validation/configuration.rb | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/lib/decision_review/utilities/pdf_validation/configuration.rb b/lib/decision_review/utilities/pdf_validation/configuration.rb index 80f93c24878..75961e382b0 100644 --- a/lib/decision_review/utilities/pdf_validation/configuration.rb +++ b/lib/decision_review/utilities/pdf_validation/configuration.rb @@ -4,7 +4,7 @@ module DecisionReview module PdfValidation class Configuration < DecisionReview::Configuration ## - # @return [String] Base path for decision review URLs. + # @return [String] Base path for PDF validation URL. # def base_path Settings.decision_review.pdf_validation.url @@ -17,6 +17,19 @@ def service_name 'DecisionReview::PDFValidation' end + ## + # @return [Hash] The basic headers required for any decision review API call. + # + def self.base_request_headers + # Can use regular Decision Reviews API key in lower environments + return super unless Rails.env.production? + + # Since we're using the `uploads/validate_document` endpoint under Benefits Intake API, + # we need to use their API key. This is pulled from BenefitsIntakeService::Configuration + api_key = Settings.benefits_intake_service.api_key || Settings.form526_backup.api_key + super.merge('apiKey' => api_key) + end + ## # Creates the a connection with parsing json and adding breakers functionality. # From 0a3f02a3c0f71fe10cbe585208cb67b1237ef77e Mon Sep 17 00:00:00 2001 From: AJ Magdub Date: Tue, 26 Mar 2024 08:33:47 -0600 Subject: [PATCH 2/5] refactor(vaos): removed clinic stop code filtering feature toggle (#16082) va.gov-team#78770 --- config/features.yml | 4 ---- modules/vaos/app/services/vaos/v2/systems_service.rb | 8 ++------ 2 files changed, 2 insertions(+), 10 deletions(-) diff --git a/config/features.yml b/config/features.yml index 28bfd610610..1d75967a58b 100644 --- a/config/features.yml +++ b/config/features.yml @@ -1020,10 +1020,6 @@ features: actor_type: user description: Allows veterans to cancel VA appointments enable_in_development: true - va_online_scheduling_clinic_filtering: - actor_type: user - description: Allows clinic selection filtering by stop codes - enable_in_development: true va_online_scheduling_community_care: actor_type: user description: Allows veterans to submit requests for Community Care appointments diff --git a/modules/vaos/app/services/vaos/v2/systems_service.rb b/modules/vaos/app/services/vaos/v2/systems_service.rb index 7b8da35423a..32503b2e1a1 100644 --- a/modules/vaos/app/services/vaos/v2/systems_service.rb +++ b/modules/vaos/app/services/vaos/v2/systems_service.rb @@ -3,8 +3,6 @@ module VAOS module V2 class SystemsService < VAOS::SessionService - STOP_CODE_FILTERS = :va_online_scheduling_clinic_filtering - def get_facility_clinics(location_id:, clinical_service: nil, clinic_ids: nil, @@ -21,11 +19,9 @@ def get_facility_clinics(location_id:, 'pageNumber' => page_number }.compact - # 'clinicalService' is used to retrieve clinics for appointment scheduling, + # 'clinicalService' is used when retrieving clinics for appointment scheduling, # triggering stop code filtering to avoid displaying unavailable clinics. - if url_params['clinicalService'].present? && Flipper.enabled?(STOP_CODE_FILTERS, user) - url_params.merge!('enableStopCodeFilter' => true) - end + url_params.merge!('enableStopCodeFilter' => true) if url_params['clinicalService'].present? response = perform(:get, url, url_params, headers) response.body[:data].map { |clinic| OpenStruct.new(clinic) } From 7f0a6ce7b79fd1f6a0797a7514d5a364aef2bbda Mon Sep 17 00:00:00 2001 From: Kristen Brown <11942904+kristen-brown@users.noreply.github.com> Date: Tue, 26 Mar 2024 11:10:01 -0400 Subject: [PATCH 3/5] Remove Job Cancellation Code from `VBADocuments::UploadProcessor` (#16087) * Remove job cancellation code which we believe is causing submissions to not be processed * Remove references to UploadProcessor cancel method --- .../sidekiq/vba_documents/upload_processor.rb | 16 ---------------- .../spec/request/v1/uploads_request_spec.rb | 1 - .../spec/request/v2/uploads_request_spec.rb | 1 - .../spec/sidekiq/upload_processor_spec.rb | 1 - 4 files changed, 19 deletions(-) diff --git a/modules/vba_documents/app/sidekiq/vba_documents/upload_processor.rb b/modules/vba_documents/app/sidekiq/vba_documents/upload_processor.rb index 43a35c36ffb..6909fec42f0 100644 --- a/modules/vba_documents/app/sidekiq/vba_documents/upload_processor.rb +++ b/modules/vba_documents/app/sidekiq/vba_documents/upload_processor.rb @@ -21,8 +21,6 @@ class UploadProcessor sidekiq_options unique_for: 30.days def perform(guid, caller_data, retries = 0) - return if cancelled? - response = nil brt = Benchmark.realtime do # @retries variable used via the CentralMail::Utilities which is included via VBADocuments::UploadValidations @@ -46,20 +44,6 @@ def perform(guid, caller_data, retries = 0) response&.success? ? true : false end - def cancelled? - Sidekiq.redis do |c| - if c.respond_to? :exists? - c.exists?("cancelled-#{jid}") - else - c.exists("cancelled-#{jid}") - end - end - end - - def self.cancel!(jid) - Sidekiq.redis { |c| c.setex("cancelled-#{jid}", 86_400, 1) } - end - private # rubocop:disable Metrics/MethodLength diff --git a/modules/vba_documents/spec/request/v1/uploads_request_spec.rb b/modules/vba_documents/spec/request/v1/uploads_request_spec.rb index aec54da50f3..7553ff67406 100644 --- a/modules/vba_documents/spec/request/v1/uploads_request_spec.rb +++ b/modules/vba_documents/spec/request/v1/uploads_request_spec.rb @@ -135,7 +135,6 @@ @md = JSON.parse(valid_metadata) @upload_submission = VBADocuments::UploadSubmission.new @upload_submission.update(status: 'uploaded') - allow_any_instance_of(VBADocuments::UploadProcessor).to receive(:cancelled?).and_return(false) allow_any_instance_of(Tempfile).to receive(:size).and_return(1) # must be > 0 or submission will error w/DOC107 allow(VBADocuments::MultipartParser).to receive(:parse) { { 'metadata' => @md.to_json, 'content' => valid_doc } diff --git a/modules/vba_documents/spec/request/v2/uploads_request_spec.rb b/modules/vba_documents/spec/request/v2/uploads_request_spec.rb index 4a4e2646cb9..5d74d97bdbd 100644 --- a/modules/vba_documents/spec/request/v2/uploads_request_spec.rb +++ b/modules/vba_documents/spec/request/v2/uploads_request_spec.rb @@ -202,7 +202,6 @@ @md = JSON.parse(valid_metadata) @upload_submission = VBADocuments::UploadSubmission.new @upload_submission.update(status: 'uploaded') - allow_any_instance_of(VBADocuments::UploadProcessor).to receive(:cancelled?).and_return(false) allow_any_instance_of(Tempfile).to receive(:size).and_return(1) # must be > 0 or submission will error w/DOC107 allow(VBADocuments::MultipartParser).to receive(:parse) { { 'metadata' => @md.to_json, 'content' => valid_doc } diff --git a/modules/vba_documents/spec/sidekiq/upload_processor_spec.rb b/modules/vba_documents/spec/sidekiq/upload_processor_spec.rb index c032d7aa89c..5c6294272eb 100644 --- a/modules/vba_documents/spec/sidekiq/upload_processor_spec.rb +++ b/modules/vba_documents/spec/sidekiq/upload_processor_spec.rb @@ -72,7 +72,6 @@ end before do - allow_any_instance_of(described_class).to receive(:cancelled?).and_return(false) allow_any_instance_of(Tempfile).to receive(:size).and_return(1) # must be > 0 or submission will error w/DOC107 objstore = instance_double(VBADocuments::ObjectStore) version = instance_double(Aws::S3::ObjectVersion) From 885d782a1f124364ed3d53e39c28fc632355134c Mon Sep 17 00:00:00 2001 From: Austin Covrig Date: Tue, 26 Mar 2024 11:26:42 -0400 Subject: [PATCH 4/5] Update test cert (#16077) --- .../claims_api/spec/fixtures/test_client.p12 | Bin 4248 -> 4339 bytes 1 file changed, 0 insertions(+), 0 deletions(-) diff --git a/modules/claims_api/spec/fixtures/test_client.p12 b/modules/claims_api/spec/fixtures/test_client.p12 index a253a22f3eda4d9ef2e6d4ff15a0e5800cbe0f5d..48fde3d8efab2dbbec58bda1c493cdb90c89613c 100644 GIT binary patch literal 4339 zcmai&Ra6uV)5qB*7Zw5OMvxX*8gZpT8YGmI*rh=iX#}K88l-CllwL~0rIB1(LO_@9 z7Nl`s-}8Ovc`o0JIdkU!znHn2-<*MjQXT>U_>fS_5+aa5qVP3ey4gKp&*SAP4f`e`|KOsFDBlBUP{M ztQ*_SC$tZc;VpgFqyk_%i$Y<7Gpxj)d;jxLD`oQ&)W)9n@K%?2?vsi*o#g$dEjuJT&uA84&J&2OfR0?W-WYHP3`>h;>~M(SxtD#csu7*n8Rqhp!HR^`4HJni3n z-guRcK5&(~`QG(MjGT5gHZ4hUn^uMw8&%-B;$YBxX5u(2aWr#@#6BLaUDDs{*Rn7!yU-;<;YO^f(AcacGXzv{pjfP4N^zW69f6zHth z-2at9k*)R3A4U?jsB*POLF87iCJ-&OhkrY_u%_DQ_k)$ zAwbK(qRXHU()_t@T{`~K?xJpIVMpz|QsHi_F|}HU&2kNreRZyI9%T887O9vUCSaNX zu+;^Gy3XL`@7HKs6$dGA-!l-nn_m|u@3Sbdp@e%C(hHhAGsfz5WE4tU-oSt_t=L1r z(v|tHI7M}T%toVu?ZE-Foq}8`gK_fCuBY| zPDKxB45KgNH|6ppGIANcP4^;*NP^d%Q(}C*j0eI!Ct4-3P&<6iu!%)bRl$5Q^#@Fm;4Nw*{CUslT ztlJZMigg*_Q%Dj<0THAlooG_BLqPK!*Wk{>WO;0>s=+_@ zeY^F0GGF;}4l>9Kym#f6cfUNv*=YZlD3I57QTJspkdobG)r6U15X~^8h;Kic=Dvi{ zVLLXwBXb!(IDUXR92=qTuoVnjUaCe{VIJ2oB6bih!YHX zx;FN+A!1^UG+T>69~Lwk&4hCHY%9XHkrVY^h2uTcuXqDlgXHadaIBF2M z#~f;r`KGXd)kYh6YP41+w4LjLW65$CaV5)1;x*HVIjxW(6f zwy~Qru0y?|QBzp&f3m`4xs+iRnj(8SdYnG^&T{qO3o}yWVxB61)@mVQ^HwEuRIAEk zKB)ietaww=OzRkF>fym zM@Qy!5W3H2J%;0_^j~E<^~@~Imc2=0wSZU4A#2YDMvg{BRnz8Ob2@%qj(+b8q!K$6 zy2JWV@8T_5_K1Li#Rn2f3jT*9{{<81mLL>#2?+)L{u{?cWdF}BFew1|*KGMKe*Rw| znix}O1-_QApnX8Yn5e>Z{69g|?e@wbKl!1)#}|7Uo(SoZzZ2-A=Y8C>1s@74c}Q{k z1)CtaOgc6ZX;G5h(e_SXt*?bEgS$uI7h+e((6Rge%5OG|;#i|HH5R0D-coO*f0Sha*ns0vffp%LI~qo0zfCoxFnMB8c&c8yN4Y&4 zJqO-{&PNsKPOBIN;zG` zuE?3JQ->J9sV6?%Ij5oO976E}4JKaE>Lm`SExk{D4>OCkr}~P7Ud!h=s=8p^D#Dv) z?%(@NJsj#ewAwZ$uEoqd1zlg)B?6^e1hG};={wP?nwMTvGwO5~x1?S{jTu5dRAk7g zKM%7L_)QU^3jNxeF9pl&kD;`_RA?ApJvD)j55Q)kBpN(MA4Va002e$ zyTLy(E@B?~($cJTF_cg@FjsucS>a}7Hr z%oa>|*ax?4hYqc`3Rw5$Xvv@TM*My}NT*YgyajX=$z5=qz< zXWG=8F=hm1aW4(*pgn)2lVzq32@dZl?tKdcm|N)fnLu9A*`=uo@AcL+j`HZQov`aJ zL)dlEpZfFDKw?+Gm>;hJW!Ilkb(v-M47{sIX7fOHtE93EyWH2BJ(rW#XW3W4=)qtSA8#%8C7|?l*-~^k&u1j3lOEp`FP>vtNXb#ik~8gHx7@?<>plH*Ollk@Ri( zcan0S`YaS4#iD@DXbh}&wek50qU=cgF??vShSScoL?~$Ssi~PgNP4kO_T(zaM zC-H>K#$w-a)#?JG6|U5Km&KFitlH+=wQ*HS4Oe)d!rCBB;Rt8lj-$^;pznv~t>U>H zWHDEGXJ#*2F~qyP7F$#l0@&xymCM?aHj6&ler^|(6kMy|r*9FbQ>{H4V`ZRb==8(~ z`MOpwE-4hAr<#CwDO2;qRq~c*Uq?}sAJzGU47~wB3dXwcqkp%4Du<~q{XD{xt7wH@ zPS#epJ#^|M+kPeXP0$xdN9*=DqmG*Q{pO~&Djs&P=#2CVZjbD6v6XP6^t8KUy*w&= z$)*-~yIQStA-YV^9jAl*aFF<5%+MRH8k^9b)rS1H|gHL!q?~fA*sqA>2utpkqUnV)vqTtOCJ(2pP?mO7@T!tfVtt5N-nb@MDu!tBj;8&Eh#aKNu zI$X}3J9#XGGmHivueelNY!eV-_bWyoei9f`gxGhNFQXpT0$5Vp$qL)kq~o6FVz_>V zEwIUd#_S1}GnH0vrS{GV=$Ia6x2ivD3t}BD8!5jmUOVB|M&$0eJ1jLJ#pq;DRHNU7 z4e{gJ7xPnvBrHAM_W-rCKM8iMlmXNH8h+Y{fVM#DcU_-7NEI|6#M@N$P_>xe`?Bmj z`y-4)q!ilkHv;CvAQL}Q@yy+?f+a9Od<$({)=1qg;#R;vU1hXM%A#-Ch!IUf*zUMI zU(UgWC$hxOY?+Tlj`Tm$ZblB8KMfA)#B@wFjW}wu70}1fUhtY(d!OVefy{?m7Txvx zgT@afa$N$bWv9>3J1cW<1@eSibYba*d^F=qG%tPo5;*q6CYDi5$m zb~ExUzE;SA&67<|e4G{>D;E#K_@zqd>uDG)S7pge#~;SBH$MI&mPa$ZQxF%+`Y6u&k+t??3m4_G)QL7T4yRtF z*on7%(q88j^rigNpODVGY+XqLw%*^luH6MjXFWs+nEgWTZJk=i0|bpW<5Yw!ODeK9 zmUD}CWv*oE+)KLloA2!2C10(w&vxh;^=MqpW3MtW!dPD?7ZzTqH|^waYvugHz#1Hw zrCcF3I$D^0MaOWCf- z-_|n}T!JltSQ>@2$PD3r720l53WE2rSTSa?BQu@C*^vl5FpnUuF{wo)qhOn;rOuD% z_JgX6l0#sXXO9>|XWzpWt{i0GDltUbaJ>Cw!BHa?mGwqXJ0@}PfYPBb_wa8^!@JQD z6L(v&=U@Qdy7pwgJItHuxZmrJ!(Xf8@svH zIAclSK7ZZi%0+baYfk|ft!}TA5S5f0dzUP;1Ng)^MRb38{2I$O8+G_X2_gwa);{U= zV{#;T;n%`yKt0(b;TXJi*U%tpi*PVRTK+Rgor}O|9KMt0fcx29OBAP+C0-8@pudNpWE3%Eu~pAI+Yhh*DhY}-36&B T{URVi$lEM-Ro=Gz_tg6j5CtT- literal 4248 zcmY+GbyO4nyT`Y&4Mum$29g58hEgJpz?24wNjbV}bUHevM7kS6V1Sgg(kLLUz!#8` zMkxWW-*fK0zkC1qoada^`*}Y9zTQYYbs7jrgv3*0pfG_*wa7CHATh82PyGapr&j%o zm5_L-;eR4%0UoOM7b$~)fWJfLKM4?d3rhOmKac~VNJ=otv%IzZJ^Dw3AP@;46c44& z;`*W`D^gmfczCz;ArHhIl`0>TMjPta*aX||qCigvmb#aQzj}8%HW|2_?!c>ebKuPQ z`&acNxX`xT9>7bxd!giKZLd-Sr4j??&#W%GVSQtvSJaa!{Gi%4*nCaoO=`f=Ny?D7 z6*+uXs*faM-tiyzbpLo*&0TnZ?}%!8+=vF3RYs4yKhz<$ls#B$rxb!e8aejd={Ifa zRCW70{1k5BPZj!Z-YGDltWX5)3Jy%6`4KK|*tuUR2)A1WrEc~WH9UOYc|P7#tGQre zvxu#o$cBz~v~<5!KDa#BKyzFXH-v6k5yhgGJ{+ayiQTS`#{-y*Wh}Vjsb;B;S0_u) zP{IR-V>Y`JR1-XrX5lVw&G@ggY$#EhlYB&iF+lJ9)j)#YhVuAWZhCY6@ov+J?)6Lj z1jXj>yZb%2?+v5?2RU1}9&{*a%8eno_l4_CHyvqKH%RTAbI?WFZiUQ&QmRHR=H2uJ zeW$w-pmj_$01yyT!9{x^cu=Po@7m&>hH~m+l~vjmW)V@=sLG+@2%pLd)M4Y(|7P1D zY?$#g7l3uvZV!54L6_kEjPes5vy7C1)%81J26lUv6fw*}J2!H9R-qyI&SOylT6FnKmG4O>&!5{pI8dhkcJh z&xgki)Wv2Wl5JXKl06D~kS0kQfLM8FB11n|DA3HIM79myX$xEYa8BW+3)ZB^qK&kO zi}#({s02IK490#XN2;Yt9T9U*y6Z;+wF#oY_@g5(F@iIT6W?#gC@4xW)6Q~v8`b>P zsU7kw5$~FX!hJ4r=MPyTC0_^+*|rGhqrk0!mE+3G4{(y$hv@Oz5aLNAUi}U{0r6?% z*W;U^p&yS-YL;`C6$w(hx-VLcUqaF9PgqS=Glsc5B*1@)d%R>XHjd0I#cIl;E4OFW zHWJ4P zo>_gq0D|X849g52)?X@E8;Ny9NfIyMOsqi7yBRy`^WICEnC)ASKam>i{?q=p;m$T? zq7-99V^=St@OXc)K8x*{9I-2f#E9k*>{{AJe>qewTmFosO6T=r>-oA?laDi_c&Hg% zAwwmjU1B?s+X9yNWHwqzd?0(2JmmDcf*MYZy)Emq?5i^Q`s*sz?;er zrQ-Lpc9(#NmYh+nR}bca-Rj7EWEft|p$+OKxciXcQm!x}vKnU_jczBH@yQu5FuLZG@mB|w?>Q1;X2w3ct>pKV-m~rNla?U zD%iaWY*~?gq2lZpp3+x4&mQNtRHb0RwjQ3XVb!a+(B~k*t+Nb-4~CHTDhg+R>Ks>x z+OEhb%uM%viDlyF( z#|2mcO4XfZJcwBB(|L2tl|rGj@%JUB{VriPJf?d(Us&_g&-`sbq@9AL;UA@*fV&tr z_&F|*9YdOad4{p_UI_Y<06Kam&AOqJ3JrvS$~&|oD+NuSKSu&q?*0DK!wwo3nWe{_ z64NhuB=|kV3`KI*YHr0b($;6N}SW?1L*oMhh`FgN$pM`Ck1)yiX3qo?r~SF zVkAxe*dS;tRz6Mc>|qso)~fR2rl{W9hQvi*UA3=R?aM=uaM8I}bP}nG2!oqu38jL0 zaUjIa#E$z<<5ki67fK;6{tqtgWp)YoMizh1|Iac+|61m(7h5cC85;kuW&WP4JtG+pWE~V}?B*-C zk6b!^uhwtf(JA%4EXBn6f!|Q(MnY}-dEpBUZm3e`i(#G4uEkV#{paZ2L6|N7DU5}q z+fDyd{iqq?9=Hy2S^p3zAN_^C#h{Th*V%}OXD*qE*E3`<+B0Yg$JPjthdc(fc|#r z$1ZIHg^>4esH8q@3ehlzwN53)-9VZLKC5Iyx_B%mWsmBs^4SpU?A7yBAH4Y!%(v$_ zqAD~RE3lvxX;^63UN3n=A`VnjBmB7}xrdGJ8Yx(>k9p6~gbqu1{=nu)r~0?sPqQ@y5X1gS)rOIjkMscj5E=7k>r5>AVYgyQVO>(ZY24f z2LiQ%+5R`&~Zg;38e!V=KW8yiJM)maKGKc_)upRE1 zE7myWiM;4KO-r$nq+jtdW0jy!zG>>$2Xs4Q*JDpQ$y(zjr|Q(BaYaW*)NupU+C;d_ zKSX@5r=)I>MK_$n7Y#+yB-T!P_C6Z9vOL&%`L%Ad)4}^Y3L=Tw!Huc;SwCs2)?_u> zbfuZ;6gg1?Wa&IulMXKx^6xr6(m#NLiSd6HF=uW;AktwNNwdrpSRn(a{!8 z(1cZI!P!%P6eP{i&we^m6VA$f?#)tW(c_=o_e}pO zy_Yuq_57K!0(C27aG3p5Z{A}ps~-O+Ljt_Ae5fq%)=PVS_wlgu-xFDu&6I5)-|!~xzYnEC1M?+R z=*=UT?ngayHEoF7ahJ+Wyh3a7-hsqvW4HxwO(e&Y?M5X1QkXw}!~YzgliX^u4UtgyTp1NQL|+F;+m(;y`hf%bZHQFM5( z4#4HFB>8sDca)N;VGqqx?3T`2}|DKCjnjG>LR&AfA zH_ZKUb_Stngu=(s3C9hTR37aX-mJvXSnKk@y7COKk86gi=~!O2h6ECCdilYvRG%Y{ zT$xLF8m>_|-#$X?bXgP7vV~7y1gJ&Yq*g6;bYb$1tkxb)6wr?7nvo4ojAGus&z}K+`HY5VU+9K2TCnx(+2wto9&LhL z^PsYYweBvi;4ET8W;5?pS}+)#3KZ*`uPPpN8y8zhcYD0lhilwCxSjCwN1B!nNuSNv z&(&Y$h8Ak(?@*o7ZV8q2oL`2Gsnsjkh@nW3_EgDhq%hNWINuUOob)VqxnWlOQX-7R3Lu}3W{&;pBLgKI}tUQG0xJQ zwRpN+RiXvc=+bh6b3`9XM+%&u_#q4WQ0rgI+Yzo|t-hrFlC{*{UsXtStF{F348)@s zwZY|gADeHazY!Ua3aL!a9tX|e>QarLn|t*;mW3sqEYRf?k&q8mer)o&!3>>aepkK45iG!Y)Q zZL&pV)2bm3y63i`1On6Nv3VMzDm!f3KBb4s-*OUJ)^L*l?WPFbZZocV`I7#%$wcES zTSrzO(?1X1eIfya7Y1H10$-qg;$~VLE0bW)?6FOG#XXPS7O20@p_6|rMLT{=%Ps0P zDy6RN(^Kf{X!cqx$*tkWOdl~k^VGSImVdc*ix#aqv#_wp+bwcL)`N?(sY@#e`9ftECsRz`VbW3dbhbRO^Kj0%(nL()`Isdnv!RS%+a9V&pz!~5Ma0Pe(Z~#Yu zHNX$a^`8|66eE~^muvBH4b<4`XppP0EUv|3#I-mUi9&KBVNi%5B@u`f0szzBRD3D4 h_NqsTI~1^}Kd;cC!)M!oNoc5!zsBk^`2Abt{tM6@&#(Xh From de101d0b0a1a0d3a42b1ecf92d4cf8993c8abb4d Mon Sep 17 00:00:00 2001 From: Austin Covrig Date: Tue, 26 Mar 2024 11:27:03 -0400 Subject: [PATCH 5/5] Match BGS with port (#16075) --- modules/claims_api/lib/bgs_service/local_bgs.rb | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/modules/claims_api/lib/bgs_service/local_bgs.rb b/modules/claims_api/lib/bgs_service/local_bgs.rb index 9e99d25c9cb..a86683900b3 100644 --- a/modules/claims_api/lib/bgs_service/local_bgs.rb +++ b/modules/claims_api/lib/bgs_service/local_bgs.rb @@ -33,8 +33,11 @@ def self.breakers_service url = Settings.bgs.url path = URI.parse(url).path host = URI.parse(url).host + port = URI.parse(url).port matcher = proc do |request_env| - request_env.url.host == host && request_env.url.path =~ /^#{path}/ + request_env.url.host == host && + request_env.url.port == port && + request_env.url.path =~ /^#{path}/ end Breakers::Service.new(