Skip to content
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.

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

IPFM-2669 fix framework generation #25

Open
wants to merge 4 commits into
base: connect
Choose a base branch
from

Conversation

rayray
Copy link

@rayray rayray commented Oct 3, 2024

IPFM-2669

Recently the ApolloTestSupport podspec changed dependency from Apollo/Core to just Apollo. Cocoapods treats the Apollo dependency as a separate entity from Apollo/Core despite them containing the same sources.

In TSM, because we use both Apollo and ApolloTestSupport, we get duplicate generated frameworks in intermediates folder that confuse the linker at build time.

By changing the ApolloTestSupport dependency back to Apollo/Core, the dupe framework is no longer generated.

calvincestari and others added 4 commits August 26, 2024 16:46
# Conflicts:
#	Sources/ApolloWebSocket/DefaultImplementation/WebSocket.swift
… instead of just Apollo, this change reduces the number of frameworks generated upstream when a Podfile uses both Apollo and the test support
version: 2.1

orbs:
secops: apollo/[email protected]

Choose a reason for hiding this comment

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

Cycode: Insecure CI/CD pipeline configuration issue: 'CircleCI workflows use uncertified CI/CD modules'.
Severity: Medium

Description

Enable this policy to be notified if your CI/CD workflows use reusable modules that are not certified by the service provider or created by a verified partner.

Cycode Remediation Guideline

Restrict - Do not allow the use of uncertified modules in this workflow, or in any workflow of this repository.
After this action has been applied, the workflow cannot run anymore, and new uncertified modules cannot be used.
Accept and Control - Map out the different modules that are used by workflows and evaluate their risk by examining their creator credibility, usage context, version etc.
To do this, use Cycode Knowledge Graph.

Tell us what to do with one of the following hashtags:

Tag Short Description
#cycode_ignore_insecure_pipeline_violation_here Applies to this resource for this violation in this request only

name: Close and Comment PR
runs-on: ubuntu-latest
steps:
- uses: superbrothers/close-pull-request@v3

Choose a reason for hiding this comment

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

Cycode: Insecure CI/CD pipeline configuration issue: 'GitHub workflows use uncertified CI/CD modules'.
Severity: Medium

Description

Enable this policy to be notified if your CI/CD workflows use reusable modules that are not certified by the service provider or created by a verified partner.

Company Remediation Guideline

The detected GitHub Action is using modules/components not certified by GitHub. It is provided by a third-party and is governed by separate terms of service, privacy policy, licensing, and or support.

Please use a certified GitHub Action, or consider writing your own to achieve the desired workflow.

For more information about writing your own GitHub Action(s), please see https://docs.github.com/en/actions/creating-actions

Please see the company guidelines here: https://thescore.atlassian.net/wiki/x/hwAPCQE

Tell us what to do with one of the following hashtags:

Tag Short Description
#cycode_ignore_insecure_pipeline_violation_here Applies to this resource for this violation in this request only

github-token: ${{ secrets.PROMETHEUS_PROJECT_ACCESS_SECRET }}
- name: Set project variables
if: ${{ success() }}
uses: austenstone/project-update@v1

Choose a reason for hiding this comment

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

Cycode: Insecure CI/CD pipeline configuration issue: 'GitHub workflows use uncertified CI/CD modules'.
Severity: Medium

Description

Enable this policy to be notified if your CI/CD workflows use reusable modules that are not certified by the service provider or created by a verified partner.

Company Remediation Guideline

The detected GitHub Action is using modules/components not certified by GitHub. It is provided by a third-party and is governed by separate terms of service, privacy policy, licensing, and or support.

Please use a certified GitHub Action, or consider writing your own to achieve the desired workflow.

For more information about writing your own GitHub Action(s), please see https://docs.github.com/en/actions/creating-actions

Please see the company guidelines here: https://thescore.atlassian.net/wiki/x/hwAPCQE

Tell us what to do with one of the following hashtags:

Tag Short Description
#cycode_ignore_insecure_pipeline_violation_here Applies to this resource for this violation in this request only

@rayray rayray changed the base branch from master to connect October 3, 2024 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants