Avoid NullPointerException when publishing SNS messages to phone numbers#7448
Merged
Avoid NullPointerException when publishing SNS messages to phone numbers#7448
Conversation
…lish request just contains a phone number
…oid NPE when only a phone number is provided
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 46 metrics, 17 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.39.0-SNAPSHOT~562b65d7bc, baseline=1.39.0-SNAPSHOT~fcd4bc4862
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.046 s) : 0, 1045805
Total [baseline] (8.487 s) : 0, 8487342
Agent [candidate] (1.045 s) : 0, 1045347
Total [candidate] (8.51 s) : 0, 8510076
section iast
Agent [baseline] (1.178 s) : 0, 1177695
Total [baseline] (8.981 s) : 0, 8980613
Agent [candidate] (1.178 s) : 0, 1177674
Total [candidate] (9.0 s) : 0, 9000366
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.175 s) : 0, 1174873
Total [baseline] (8.98 s) : 0, 8979632
Agent [candidate] (1.174 s) : 0, 1174127
Total [candidate] (8.965 s) : 0, 8964950
section iast_TELEMETRY_OFF
Agent [baseline] (1.18 s) : 0, 1180496
Total [baseline] (8.965 s) : 0, 8965005
Agent [candidate] (1.193 s) : 0, 1193101
Total [candidate] (9.057 s) : 0, 9057070
gantt
title insecure-bank - break down per module: candidate=1.39.0-SNAPSHOT~562b65d7bc, baseline=1.39.0-SNAPSHOT~fcd4bc4862
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (667.2 ms) : 0, 667200
BytebuddyAgent [candidate] (667.453 ms) : 0, 667453
GlobalTracer [baseline] (306.104 ms) : 0, 306104
GlobalTracer [candidate] (305.413 ms) : 0, 305413
AppSec [baseline] (50.955 ms) : 0, 50955
AppSec [candidate] (50.901 ms) : 0, 50901
Remote Config [baseline] (678.551 µs) : 0, 679
Remote Config [candidate] (679.59 µs) : 0, 680
Telemetry [baseline] (7.44 ms) : 0, 7440
Telemetry [candidate] (7.482 ms) : 0, 7482
section iast
BytebuddyAgent [baseline] (782.426 ms) : 0, 782426
BytebuddyAgent [candidate] (783.772 ms) : 0, 783772
GlobalTracer [baseline] (297.705 ms) : 0, 297705
GlobalTracer [candidate] (296.271 ms) : 0, 296271
AppSec [baseline] (53.092 ms) : 0, 53092
AppSec [candidate] (52.235 ms) : 0, 52235
IAST [baseline] (22.406 ms) : 0, 22406
IAST [candidate] (22.479 ms) : 0, 22479
Remote Config [baseline] (584.942 µs) : 0, 585
Remote Config [candidate] (573.18 µs) : 0, 573
Telemetry [baseline] (7.99 ms) : 0, 7990
Telemetry [candidate] (8.825 ms) : 0, 8825
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (780.616 ms) : 0, 780616
BytebuddyAgent [candidate] (781.024 ms) : 0, 781024
GlobalTracer [baseline] (296.183 ms) : 0, 296183
GlobalTracer [candidate] (296.308 ms) : 0, 296308
AppSec [baseline] (49.297 ms) : 0, 49297
AppSec [candidate] (48.717 ms) : 0, 48717
IAST [baseline] (21.948 ms) : 0, 21948
IAST [candidate] (24.5 ms) : 0, 24500
Remote Config [baseline] (591.768 µs) : 0, 592
Remote Config [candidate] (599.463 µs) : 0, 599
Telemetry [baseline] (12.749 ms) : 0, 12749
Telemetry [candidate] (9.46 ms) : 0, 9460
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (784.579 ms) : 0, 784579
BytebuddyAgent [candidate] (793.223 ms) : 0, 793223
GlobalTracer [baseline] (298.55 ms) : 0, 298550
GlobalTracer [candidate] (300.753 ms) : 0, 300753
AppSec [baseline] (52.19 ms) : 0, 52190
AppSec [candidate] (52.323 ms) : 0, 52323
IAST [baseline] (23.148 ms) : 0, 23148
IAST [candidate] (22.83 ms) : 0, 22830
Remote Config [baseline] (615.119 µs) : 0, 615
Remote Config [candidate] (642.002 µs) : 0, 642
Telemetry [baseline] (7.814 ms) : 0, 7814
Telemetry [candidate] (9.594 ms) : 0, 9594
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.39.0-SNAPSHOT~562b65d7bc, baseline=1.39.0-SNAPSHOT~fcd4bc4862
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.055 s) : 0, 1054614
Total [baseline] (10.408 s) : 0, 10407937
Agent [candidate] (1.055 s) : 0, 1055178
Total [candidate] (10.321 s) : 0, 10320567
section appsec
Agent [baseline] (1.169 s) : 0, 1169133
Total [baseline] (10.417 s) : 0, 10416843
Agent [candidate] (1.174 s) : 0, 1173512
Total [candidate] (10.536 s) : 0, 10535538
section iast
Agent [baseline] (1.186 s) : 0, 1185566
Total [baseline] (10.851 s) : 0, 10851239
Agent [candidate] (1.185 s) : 0, 1184672
Total [candidate] (10.871 s) : 0, 10871239
section profiling
Agent [baseline] (1.248 s) : 0, 1248020
Total [baseline] (10.543 s) : 0, 10542593
Agent [candidate] (1.244 s) : 0, 1244397
Total [candidate] (10.551 s) : 0, 10550986
gantt
title petclinic - break down per module: candidate=1.39.0-SNAPSHOT~562b65d7bc, baseline=1.39.0-SNAPSHOT~fcd4bc4862
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (673.272 ms) : 0, 673272
BytebuddyAgent [candidate] (673.542 ms) : 0, 673542
GlobalTracer [baseline] (308.304 ms) : 0, 308304
GlobalTracer [candidate] (308.431 ms) : 0, 308431
AppSec [baseline] (51.236 ms) : 0, 51236
AppSec [candidate] (51.416 ms) : 0, 51416
Remote Config [baseline] (692.148 µs) : 0, 692
Remote Config [candidate] (708.289 µs) : 0, 708
Telemetry [baseline] (7.551 ms) : 0, 7551
Telemetry [candidate] (7.53 ms) : 0, 7530
section appsec
BytebuddyAgent [baseline] (678.2 ms) : 0, 678200
BytebuddyAgent [candidate] (680.256 ms) : 0, 680256
GlobalTracer [baseline] (299.723 ms) : 0, 299723
GlobalTracer [candidate] (301.253 ms) : 0, 301253
AppSec [baseline] (157.078 ms) : 0, 157078
AppSec [candidate] (157.559 ms) : 0, 157559
IAST [baseline] (21.505 ms) : 0, 21505
IAST [candidate] (21.12 ms) : 0, 21120
Remote Config [baseline] (600.189 µs) : 0, 600
Remote Config [candidate] (604.534 µs) : 0, 605
Telemetry [baseline] (9.303 ms) : 0, 9303
Telemetry [candidate] (9.738 ms) : 0, 9738
section iast
BytebuddyAgent [baseline] (789.493 ms) : 0, 789493
BytebuddyAgent [candidate] (788.995 ms) : 0, 788995
GlobalTracer [baseline] (298.762 ms) : 0, 298762
GlobalTracer [candidate] (298.444 ms) : 0, 298444
AppSec [baseline] (52.476 ms) : 0, 52476
AppSec [candidate] (52.741 ms) : 0, 52741
IAST [baseline] (21.909 ms) : 0, 21909
IAST [candidate] (21.646 ms) : 0, 21646
Remote Config [baseline] (591.14 µs) : 0, 591
Remote Config [candidate] (582.992 µs) : 0, 583
Telemetry [baseline] (8.736 ms) : 0, 8736
Telemetry [candidate] (8.653 ms) : 0, 8653
section profiling
BytebuddyAgent [baseline] (664.456 ms) : 0, 664456
BytebuddyAgent [candidate] (663.019 ms) : 0, 663019
GlobalTracer [baseline] (391.429 ms) : 0, 391429
GlobalTracer [candidate] (390.839 ms) : 0, 390839
AppSec [baseline] (52.249 ms) : 0, 52249
AppSec [candidate] (51.622 ms) : 0, 51622
Remote Config [baseline] (695.066 µs) : 0, 695
Remote Config [candidate] (684.297 µs) : 0, 684
Telemetry [baseline] (7.423 ms) : 0, 7423
Telemetry [candidate] (7.371 ms) : 0, 7371
ProfilingAgent [baseline] (94.511 ms) : 0, 94511
ProfilingAgent [candidate] (93.674 ms) : 0, 93674
Profiling [baseline] (94.536 ms) : 0, 94536
Profiling [candidate] (93.699 ms) : 0, 93699
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 16 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.39.0-SNAPSHOT~562b65d7bc, baseline=1.39.0-SNAPSHOT~fcd4bc4862
dateFormat X
axisFormat %s
section baseline
no_agent (1.327 ms) : 1308, 1346
. : milestone, 1327,
appsec (1.72 ms) : 1696, 1744
. : milestone, 1720,
appsec_no_iast (1.731 ms) : 1706, 1755
. : milestone, 1731,
iast (1.461 ms) : 1438, 1484
. : milestone, 1461,
profiling (1.488 ms) : 1461, 1514
. : milestone, 1488,
tracing (1.461 ms) : 1436, 1486
. : milestone, 1461,
section candidate
no_agent (1.325 ms) : 1305, 1345
. : milestone, 1325,
appsec (1.697 ms) : 1673, 1721
. : milestone, 1697,
appsec_no_iast (1.705 ms) : 1680, 1730
. : milestone, 1705,
iast (1.48 ms) : 1457, 1502
. : milestone, 1480,
profiling (1.475 ms) : 1449, 1500
. : milestone, 1475,
tracing (1.461 ms) : 1436, 1485
. : milestone, 1461,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.39.0-SNAPSHOT~562b65d7bc, baseline=1.39.0-SNAPSHOT~fcd4bc4862
dateFormat X
axisFormat %s
section baseline
no_agent (364.473 µs) : 343, 385
. : milestone, 364,
iast (479.853 µs) : 459, 501
. : milestone, 480,
iast_FULL (554.913 µs) : 532, 578
. : milestone, 555,
iast_GLOBAL (511.638 µs) : 489, 534
. : milestone, 512,
iast_HARDCODED_SECRET_DISABLED (479.231 µs) : 458, 501
. : milestone, 479,
iast_INACTIVE (451.423 µs) : 429, 474
. : milestone, 451,
iast_TELEMETRY_OFF (466.75 µs) : 446, 488
. : milestone, 467,
tracing (442.386 µs) : 422, 463
. : milestone, 442,
section candidate
no_agent (371.683 µs) : 352, 391
. : milestone, 372,
iast (480.348 µs) : 459, 502
. : milestone, 480,
iast_FULL (551.279 µs) : 529, 574
. : milestone, 551,
iast_GLOBAL (503.761 µs) : 482, 526
. : milestone, 504,
iast_HARDCODED_SECRET_DISABLED (483.103 µs) : 462, 504
. : milestone, 483,
iast_INACTIVE (459.306 µs) : 436, 483
. : milestone, 459,
iast_TELEMETRY_OFF (467.756 µs) : 446, 489
. : milestone, 468,
tracing (440.181 µs) : 420, 460
. : milestone, 440,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics. Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.39.0-SNAPSHOT~562b65d7bc, baseline=1.39.0-SNAPSHOT~fcd4bc4862
dateFormat X
axisFormat %s
section baseline
no_agent (1.468 ms) : 1457, 1480
. : milestone, 1468,
appsec (2.233 ms) : 2197, 2269
. : milestone, 2233,
iast (1.992 ms) : 1948, 2035
. : milestone, 1992,
iast_GLOBAL (2.03 ms) : 1986, 2074
. : milestone, 2030,
profiling (1.867 ms) : 1832, 1901
. : milestone, 1867,
tracing (1.847 ms) : 1813, 1880
. : milestone, 1847,
section candidate
no_agent (1.456 ms) : 1445, 1468
. : milestone, 1456,
appsec (2.246 ms) : 2210, 2282
. : milestone, 2246,
iast (1.987 ms) : 1943, 2030
. : milestone, 1987,
iast_GLOBAL (2.021 ms) : 1977, 2065
. : milestone, 2021,
profiling (1.877 ms) : 1841, 1913
. : milestone, 1877,
tracing (1.853 ms) : 1819, 1886
. : milestone, 1853,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.39.0-SNAPSHOT~562b65d7bc, baseline=1.39.0-SNAPSHOT~fcd4bc4862
dateFormat X
axisFormat %s
section baseline
no_agent (15.005 s) : 15005000, 15005000
. : milestone, 15005000,
appsec (15.067 s) : 15067000, 15067000
. : milestone, 15067000,
iast (18.844 s) : 18844000, 18844000
. : milestone, 18844000,
iast_GLOBAL (17.759 s) : 17759000, 17759000
. : milestone, 17759000,
profiling (15.507 s) : 15507000, 15507000
. : milestone, 15507000,
tracing (15.072 s) : 15072000, 15072000
. : milestone, 15072000,
section candidate
no_agent (15.472 s) : 15472000, 15472000
. : milestone, 15472000,
appsec (14.826 s) : 14826000, 14826000
. : milestone, 14826000,
iast (18.895 s) : 18895000, 18895000
. : milestone, 18895000,
iast_GLOBAL (17.883 s) : 17883000, 17883000
. : milestone, 17883000,
profiling (15.532 s) : 15532000, 15532000
. : milestone, 15532000,
tracing (14.846 s) : 14846000, 14846000
. : milestone, 14846000,
|
nayeem-kamal
approved these changes
Aug 15, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What Does This Do
Avoid NPE when only a phone number is provided in an SNS publish request
Motivation
Both
topicArnandtargetArnare optional for SNS publish requests: https://docs.aws.amazon.com/sns/latest/api/API_Publish.htmlAdditional Notes
topicARNis required for SNS publish batch requests: https://docs.aws.amazon.com/sns/latest/api/API_PublishBatch.html so we don't need to change that part of the code.Jira ticket: APMS-13002