fix(datastore): filter authrules with invalid ownerfield

[diegocstn]

Description of changes:
This PR introduces changes to AuthRuleDecorator to support multiple rules with different owner fields as per documentation.

Given the following schema

type PostDraftCognitoMultiOwner @model
  @auth(rules: [
    { allow: owner },
    { allow: owner, ownerField: "editors", operations: [update, read] }
  ]) {
  id: ID!
  title: String!
  content: String
  owner: String
  editors: [String]
}

AppSync generates

Check points: (check or cross out if not relevant)

• Added new tests to cover change, if needed
  - [ ] Build succeeds with all target using Swift Package Manager
• All unit tests pass
• All integration tests pass
  - [ ] Documentation update for the change if required
• PR title conforms to conventional commit style
  - [ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[maziarzamani]

Can't wait for this to be merged. Any ETA? Cheers.

[piro49]

Same over here!

[maziarzamani]

Hi @diegocstn. I know you have loads on your plate, but I have 2 projects waiting for this PR to go through and released. Can we expedite it, seems like we are 99% of the way? Cheers.

[maziarzamani]

@diegocstn, while we wait for this PR to go through, is there a temporary workaround? E.g. can the primary owner be turned into an array, so we can at least support multi-owner?

type PostDraftCognitoMultiOwner @model
  @auth(rules: [
    { allow: owner }
  ]) {
  id: ID!
  title: String!
  content: String
  owner: [String]
}

[maziarzamani]

@diegocstn, can you provide an update for this issue? It's been almost 3months, and yet this need-to-have feature is still not working on iOS.

[maziarzamani]

Almost 4months has passed and a standard feature in amplify-cli is still not supported in a major SDK. Unfortunately this will kill all my iOS related projects going forward.

[kavamara]

any further with this?

[maziarzamani]

Documentation was updated recently with something that could indicate there is support for this feature. Could there be light at the end of the tunnel?

aws-amplify/docs@1562ca1

[lawmicha]

This PR is no longer needed, it was re-done in #3223