Skip to content

Synternet pubsub examples update - #3

Merged
Dancia merged 2 commits into
mainfrom
synternet-pubsub-update-2025-01
Feb 4, 2025
Merged

Dancia merged 2 commits into
mainfrom
synternet-pubsub-update-2025-01

Conversation

@GytisNars

Copy link
Copy Markdown
Contributor

Description

Updating python package version. Renewed URL links in README file. Rework of given example scripts: changing variable names for better understanding, complete rework of "republish_using_creds_files.py" file - used deprecated scenarios so changes were made according to available actions. at Synternet Portal.

Proposed Changes

List the specific changes or additions made in this PR.

  • Python 'nkeys' package version changed from 0.1.0 to 0.2.1 (was failing on 0.1.0)
  • Updated URL links/naming in README (Updated everything to the current Synternet name)
  • Example scripts update:
    publish.py: subject naming change (for easier understanding and unification between other pubsub repos)
    republish_using_creds_files.py: complete re-do. Instead of one NATS jwt and connection usage to subscribe and publish again - switched to two separate connections/jwt usage. Synternet restrict NATS creds only to perform 1 type of action so now 1st CREDS will be used for publisher and 2nd for subscriber only

Checklist

  • I have tested these changes locally
  • I have reviewed the code for any potential issues
  • I have documented any necessary changes

Packages listed in requirements.txt were outdated -> renewed to the
latest versions. "republish_using_creds_file.py" example was outdated
and required a rework. Updates are aligned with the actions available on
Synternet Portal.

@Dancia Dancia left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove republish_using_creds_file.py before merging. Edit: nevermind, you actually updated the script.

Optionally, you could do the update you suggested:

It would be nice if NATS, ACCESS TOKEN and SUBJECT variables would be put in the first lines

@Dancia
Dancia merged commit 55e533e into main Feb 4, 2025
@Dancia
Dancia deleted the synternet-pubsub-update-2025-01 branch February 4, 2025 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants