Attention is currently required from: pespin.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35405?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified-1 by Jenkins Builder
Change subject: Split Diameter TS 29.272 templates into its own file
......................................................................
Split Diameter TS 29.272 templates into its own file
This allows more easily finding, understanding and using the tons of
templates coming from different specs.
Change-Id: I2e6b813a9e4ff98eef575aaaf46dbcf9f1a5d2d4
---
M dia2gsup/DIA2GSUP_Tests.ttcn
M dia2gsup/gen_links.sh
M ggsn_tests/GGSN_Tests.ttcn
M ggsn_tests/gen_links.sh
M hss/HSS_Tests.ttcn
M hss/gen_links.sh
M library/DIAMETER_Templates.ttcn
A library/DIAMETER_ts29_272_Templates.ttcn
M mme/MME_Tests.ttcn
M mme/gen_links.sh
M pgw/PGW_Tests.ttcn
M pgw/gen_links.sh
12 files changed, 660 insertions(+), 616 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/05/35405/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35405?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I2e6b813a9e4ff98eef575aaaf46dbcf9f1a5d2d4
Gerrit-Change-Number: 35405
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35394?usp=email )
Change subject: mme: Fix TEID values used in test
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Yeah, I meant fixing in the meaning of repairing the problem. I'd have used "hardcode" or alike instead if I wanted to use the other meaning, but I agree in this case the term can have both meanings.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35394?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I5a32491a00eb6bad505e4bcf9a88017382073138
Gerrit-Change-Number: 35394
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Thu, 21 Dec 2023 14:39:28 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/35404?usp=email )
Change subject: OBS: srcpkg Dockerfile: add lsb-release, quilt
......................................................................
OBS: srcpkg Dockerfile: add lsb-release, quilt
Wireshark with Osmocom patches on top of wireshark.git master failed to
build. I've reworked the packaging to use the "debian" dir from upstream
instead of having our own (now outdated) copy:
https://gitea.osmocom.org/osmocom/wireshark/commits/branch/osmith/deb-packa…
Add the additional dependencies that are now required to build the
wireshark source package to the Dockerfile.
Related: OS#2537
Change-Id: I82f64de70b9d099ac404530ca74c771d66b306d8
---
M scripts/obs/data/build_srcpkg.Dockerfile
1 file changed, 20 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/04/35404/1
diff --git a/scripts/obs/data/build_srcpkg.Dockerfile b/scripts/obs/data/build_srcpkg.Dockerfile
index 5475a94..80488ea 100644
--- a/scripts/obs/data/build_srcpkg.Dockerfile
+++ b/scripts/obs/data/build_srcpkg.Dockerfile
@@ -16,10 +16,12 @@
git-review \
gnupg2 \
libxml2-utils \
+ lsb-release \
meson \
osc \
python3-packaging \
python3-setuptools \
+ quilt \
rebar3 \
sed \
&& \
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/35404?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I82f64de70b9d099ac404530ca74c771d66b306d8
Gerrit-Change-Number: 35404
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: newchange
laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/34962?usp=email )
(
4 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
)Change subject: transport: Extend the documentation for each transport driver
......................................................................
transport: Extend the documentation for each transport driver
This driver description we add to the code is automatically added to the
respective user manual sections.
Change-Id: I8807bfb11f43b167f1321d556e09ec5234fff629
---
M pySim/transport/calypso.py
M pySim/transport/modem_atcmd.py
M pySim/transport/pcsc.py
M pySim/transport/serial.py
4 files changed, 28 insertions(+), 4 deletions(-)
Approvals:
osmith: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/pySim/transport/calypso.py b/pySim/transport/calypso.py
index dd30f80..7544d2e 100644
--- a/pySim/transport/calypso.py
+++ b/pySim/transport/calypso.py
@@ -167,6 +167,9 @@
@staticmethod
def argparse_add_reader_args(arg_parser: argparse.ArgumentParser):
- osmobb_group = arg_parser.add_argument_group('OsmocomBB Reader')
+ osmobb_group = arg_parser.add_argument_group('OsmocomBB Reader', """Use an OsmocomBB compatible phone
+to access the SIM inserted to the phone SIM slot. This will require you to run the OsmocomBB firmware inside
+the phone (can be ram-loaded). It also requires that you run the ``osmocon`` program, which provides a unix
+domain socket to which this reader driver can attach.""")
osmobb_group.add_argument('--osmocon', dest='osmocon_sock', metavar='PATH', default=None,
help='Socket path for Calypso (e.g. Motorola C1XX) based reader (via OsmocomBB)')
diff --git a/pySim/transport/modem_atcmd.py b/pySim/transport/modem_atcmd.py
index 88e6253..a05d6c4 100644
--- a/pySim/transport/modem_atcmd.py
+++ b/pySim/transport/modem_atcmd.py
@@ -180,7 +180,9 @@
@staticmethod
def argparse_add_reader_args(arg_parser: argparse.ArgumentParser):
- modem_group = arg_parser.add_argument_group('AT Command Modem Reader')
+ modem_group = arg_parser.add_argument_group('AT Command Modem Reader', """Talk to a SIM Card inside a
+mobile phone or cellular modem which is attached to this computer and offers an AT command interface including
+the AT+CSIM interface for Generic SIM access as specified in 3GPP TS 27.007.""")
modem_group.add_argument('--modem-device', dest='modem_dev', metavar='DEV', default=None,
help='Serial port of modem for Generic SIM Access (3GPP TS 27.007)')
modem_group.add_argument('--modem-baud', type=int, metavar='BAUD', default=115200,
diff --git a/pySim/transport/pcsc.py b/pySim/transport/pcsc.py
index 8a99e9f..a6714d5 100644
--- a/pySim/transport/pcsc.py
+++ b/pySim/transport/pcsc.py
@@ -114,7 +114,12 @@
@staticmethod
def argparse_add_reader_args(arg_parser: argparse.ArgumentParser):
- pcsc_group = arg_parser.add_argument_group('PC/SC Reader')
+ pcsc_group = arg_parser.add_argument_group('PC/SC Reader',
+ """Use a PC/SC card reader to talk to the SIM card. PC/SC is a standard API for how applications
+access smart card readers, and is available on a variety of operating systems, such as Microsoft
+Windows, MacOS X and Linux. Most vendors of smart card readers provide drivers that offer a PC/SC
+interface, if not even a generic USB CCID driver is used. You can use a tool like ``pcsc_scan -r``
+to obtain a list of readers available on your system. """)
dev_group = pcsc_group.add_mutually_exclusive_group()
dev_group.add_argument('-p', '--pcsc-device', type=int, dest='pcsc_dev', metavar='PCSC', default=None,
help='Number of PC/SC reader to use for SIM access')
diff --git a/pySim/transport/serial.py b/pySim/transport/serial.py
index fbf907f..06a082a 100644
--- a/pySim/transport/serial.py
+++ b/pySim/transport/serial.py
@@ -244,7 +244,9 @@
@staticmethod
def argparse_add_reader_args(arg_parser: argparse.ArgumentParser):
- serial_group = arg_parser.add_argument_group('Serial Reader')
+ serial_group = arg_parser.add_argument_group('Serial Reader', """Use a simple/ultra-low-cost serial reader
+attached to a (physical or USB/virtual) RS232 port. This doesn't work with all RS232-attached smart card
+readers, only with the very primitive readers following the ancient `Phoenix` or `Smart Mouse` design.""")
serial_group.add_argument('-d', '--device', metavar='DEV', default='/dev/ttyUSB0',
help='Serial Device for SIM access')
serial_group.add_argument('-b', '--baud', dest='baudrate', type=int, metavar='BAUD', default=9600,
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/34962?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I8807bfb11f43b167f1321d556e09ec5234fff629
Gerrit-Change-Number: 34962
Gerrit-PatchSet: 5
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: merged
laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/34960?usp=email )
Change subject: transport/pcsc: Allow opening PC/SC readers by a regex of their name
......................................................................
transport/pcsc: Allow opening PC/SC readers by a regex of their name
Opening PC/SC readers by index/number is very error-prone as the order
is never deterministic in any system with multiple (hot-plugged, USB)
readers. Instead, let's offer the alternative of specifying a regular
expression to match the reader name (similar to remsim-bankd).
Change-Id: I983f19c6741904c1adf27749c9801b44a03a5d78
---
M pySim/transport/__init__.py
M pySim/transport/pcsc.py
2 files changed, 45 insertions(+), 13 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/pySim/transport/__init__.py b/pySim/transport/__init__.py
index e0e1e62..d075878 100644
--- a/pySim/transport/__init__.py
+++ b/pySim/transport/__init__.py
@@ -287,7 +287,7 @@
"""
Init card reader driver
"""
- if opts.pcsc_dev is not None:
+ if opts.pcsc_dev is not None or opts.pcsc_regex is not None:
from pySim.transport.pcsc import PcscSimLink
sl = PcscSimLink(opts, **kwargs)
elif opts.osmocon_sock is not None:
diff --git a/pySim/transport/pcsc.py b/pySim/transport/pcsc.py
index 6afcdac..b1c424f 100644
--- a/pySim/transport/pcsc.py
+++ b/pySim/transport/pcsc.py
@@ -1,7 +1,7 @@
# -*- coding: utf-8 -*-
# Copyright (C) 2009-2010 Sylvain Munaut <tnt(a)246tNt.com>
-# Copyright (C) 2010 Harald Welte <laforge(a)gnumonks.org>
+# Copyright (C) 2010-2023 Harald Welte <laforge(a)gnumonks.org>
#
# This program is free software: you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
@@ -19,7 +19,8 @@
import argparse
import os
-from typing import Optional
+import re
+from typing import Optional, Union
from smartcard.CardConnection import CardConnection
from smartcard.CardRequest import CardRequest
@@ -36,16 +37,30 @@
def __init__(self, opts: argparse.Namespace = argparse.Namespace(pcsc_dev=0), **kwargs):
super().__init__(**kwargs)
+ self._reader = None
+ r = readers()
+ if opts.pcsc_dev is not None:
+ # actual reader index number (integer)
+ reader_number = opts.pcsc_dev
+ if reader_number >= len(r):
+ raise ReaderError('No reader found for number %d' % reader_number)
+ self._reader = r[reader_number]
+ else:
+ # reader regex string
+ cre = re.compile(opts.pcsc_regex)
+ for reader in r:
+ if cre.search(reader.name):
+ self._reader = reader
+ break
+ if not self._reader:
+ raise ReaderError('No matching reader found for regex %s' % opts.pcsc_regex)
+
+ self._con = self._reader.createConnection()
+
if os.environ.get('PYSIM_INTEGRATION_TEST') == "1":
print("Using PC/SC reader interface")
else:
- print("Using PC/SC reader number %u" % opts.pcsc_dev)
- r = readers()
- if opts.pcsc_dev >= len(r):
- raise ReaderError('No reader found for number %d' % opts.pcsc_dev)
- self._reader = r[opts.pcsc_dev]
- self._con = self._reader.createConnection()
- self._reader_number = opts.pcsc_dev
+ print("Using PC/SC reader %s" % self)
def __del__(self):
try:
@@ -100,10 +115,13 @@
return i2h(data), i2h(sw)
def __str__(self) -> str:
- return "PCSC:%u[%s]" % (self._reader_number, self._reader)
+ return "PCSC[%s]" % (self._reader)
@staticmethod
def argparse_add_reader_args(arg_parser: argparse.ArgumentParser):
pcsc_group = arg_parser.add_argument_group('PC/SC Reader')
- pcsc_group.add_argument('-p', '--pcsc-device', type=int, dest='pcsc_dev', metavar='PCSC', default=None,
- help='PC/SC reader number to use for SIM access')
+ dev_group = pcsc_group.add_mutually_exclusive_group()
+ dev_group.add_argument('-p', '--pcsc-device', type=int, dest='pcsc_dev', metavar='PCSC', default=None,
+ help='Number of PC/SC reader to use for SIM access')
+ dev_group.add_argument('--pcsc-regex', type=str, dest='pcsc_regex', metavar='REGEX', default=None,
+ help='Regex matching PC/SC reader to use for SIM access')
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/34960?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I983f19c6741904c1adf27749c9801b44a03a5d78
Gerrit-Change-Number: 34960
Gerrit-PatchSet: 5
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: merged