Skip to content

salesforce_message_publisher assumes location of .pem file #595

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
c-simpson opened this issue Dec 31, 2023 · 0 comments · Fixed by #596
Closed

salesforce_message_publisher assumes location of .pem file #595

c-simpson opened this issue Dec 31, 2023 · 0 comments · Fixed by #596
Assignees
@sposerina

Comments

@c-simpson
Copy link
Collaborator

Unlike API_ingest/ salesforce_contacts.py (see #594)

salesforce_message_publisher assumes a fixed location for the .pem file, but, as per #594, that's not valid.

Is this code still used? If so, maybe we should create a find_pem_file() that either can use.

def send_pipeline_update_messages(contacts_list):
    pem_file = 'bin/connected-app-secrets.pem'
c-simpson added a commit that referenced this issue Dec 31, 2023
@c-simpson
Create and use find_pem_file() #595
d30566e
@c-simpson c-simpson mentioned this issue Dec 31, 2023
Merged
@c-simpson c-simpson linked a pull request Dec 31, 2023 that will close this issue
Create and use find_pem_file() #595 #596
Merged
c-simpson added a commit that referenced this issue Dec 31, 2023
@c-simpson
Merge pull request #596 from CodeForPhilly/pem_file_finder
c839e52 
Create and use find_pem_file()   #595
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sposerina sposerina

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Create and use find_pem_file() #595 CodeForPhilly/paws-data-pipeline
2 participants
@sposerina @c-simpson