2024-04-04: Number verification / SIM Swap / OTP - Meeting Minutes

Attendees & Representation

Community: Ludovic Robert Fernando Prado Cabrillo Jorge Garcia Hospital Axel Nennker Izahir Clemencia  Ming Hui Foo Ákos Hunyadi Cormac Hegarty  @Aziz Abouchi Surajj Jaggernath 

Agenda

• Roll Call
  • Move to Confluence
    • Change to zoom coming as we have to use LF zoom.
  • Do we have a code owner candidate from DT as Dawid step off
  • Ákos Hunyadi will check in DT for next meeting 
    • If not looking for candidate.
• Action Items Review
• Number verification
• Sim Swap
• OTP

Minutes

Action Item Review

Number Verification

New

• integration to on device application (EAP-AKA) (integration to on device application (EAP-AKA) · Issue #86 · camaraproject/NumberVerification (github.com))
  • GSMA Specification released end of April
  • This is probably a topic with a larger impact than our projet - Should be discussed in GSMA Open Gateway? 
• Mandate "+" prefix in phone number
  • PR Mandate "+" prefix in phone number by fernandopradocabrillo · Pull Request #90 · camaraproject/NumberVerification (github.com)
  • Ready to be merged
• Include Authorization and Authentication section in API description according to commonalities v0.3
  • Issue  Include Authorization and authentication section in API description · Issue #88 · camaraproject/NumberVerification (github.com)
  • PR Include auth section in api description by fernandopradocabrillo · Pull Request #89 · camaraproject/NumberVerification (github.com)
  • Ready to be merged too

Ongoing

• Include linter & fix linter issues
  • Looking for Rafal Artych help
  • Akos will contact Rafal to provide support on this topic
• Adding a '+' to the phone number before hashing should be mandatory, not optional (Adding a '+' to the phone number before hashing should be mandatory, not optional · Issue #65 · camaraproject/NumberVerification (github.com))
  • Proposal
    • Make '+' mandatory in both plain & hashed as decided in commonalities (new PR 90)
    • Regarding the point about removing the hashing value, it should be tackled in another issue.
• (Action): Fernando Prado Cabrillo will check number verification version/release as it seems that we have an issue
• (Action): Ming Hui Foo will raise a PR with a proposal to update the Sequence Diagram including Aggregation and Federation models

SIM Swap

New

• Update SIM Swap & SIM Swap notification subscription yaml to make the '+' mandatory for phoneNumber (Update SIM Swap & SIM Swap notification subscription yaml to make the '+' mandatory for phoneNumber by bigludo7 · Pull Request #100 · camaraproject/SimSwap (github.com))
• Already merged

Ongoing

• Definition of test cases
  • Telefonica test case proposal (Telefonica test cases proposal by fernandopradocabrillo · Pull Request #70 · camaraproject/SimSwap (github.com)) 
  • Submission of test cases (Submission of test cases by trehman-gsma · Pull Request #68 · camaraproject/SimSwap (github.com))
  • Submission of test cases (Gherkin feature files) (Submission of test cases (Gherkin feature files) · Issue #63 · camaraproject/SimSwap (github.com))
  • no news
    • Topics discussed here: Enhancement of the Testing Guidelines → Enhancement of the Testing Guidelines · Issue #158 · camaraproject/Commonalities (github.com) → Once an agreement has been reached in Commonalities Fernando Prado Cabrillo will adapt the test proposal to align with the agreement

• PhoneNumber pattern - clarification on optional "+" prefix (PhoneNumber pattern - clarification on optional "+" prefix · Issue #85 · camaraproject/SimSwap (github.com))
  • Discussed & merged in Commonalities → Mandate '+' in all phoneNumber formats (Mandate '+' in all phoneNumber formats by fernandopradocabrillo · Pull Request #148 · camaraproject/Commonalities (github.com))

• Include linter & fix linter issues
  • Looking for Rafal Artych help

• Release plan:
  • Notification subscriptions: As discussed we'll wait for the open Commonalities issues to be resolved and then we'll align with the outcome and proceed with a release

OTP

New

• New issue to mandate '+' in phone number (Mandate '+' in phoneNumber · Issue #47 · camaraproject/OTPvalidationAPI (github.com))
  • PR Make '+' mandatory for phoneNumber by bigludo7 · Pull Request #50 · camaraproject/OTPvalidationAPI (github.com)
    • Issue with codeowners.

Ongoing

• no ongoing issue or PR

Action items

• Fernando Prado Cabrillo will check number verification version/release as it seems that we have an issue
• Fernando Prado Cabrillo will check permissions as codeowner in OTP Validation to review and merge Ludovic's PR regarding "+" prefix
• Ming Hui Foo will raise a PR with a proposal to update the Sequence Diagram including Aggregation and Federation models