Feature #8863
Add tenantName logic related to sec_authentication_domain in FWD
100%
History
#1 Updated by Radu Apetrii about 2 years ago
tenantName logic that is related to the sec_authentication_domain table in FWD.Currently,
SecurityOps.WorkArea.addDomain is the method that handles domain creation. In here, there are multiple columns set, but tenantName is not amongst them.In terms of implementation, at least for the start, the following things need to be done:
- When creating a domain, specify the tenant for which the domain is created.
- When a tenant tries to access a domain, check if the domain is valid (can be assigned to that tenant).
- In
SecurityOps.readDomains, retrieve the tenant name from thesec_authentication_domain.
#3 Updated by Eric Faulhaber about 2 years ago
- Project changed from Base Language to Runtime Infrastructure
#4 Updated by Eric Faulhaber over 1 year ago
- Assignee set to Ovidiu Maxiniuc
- Priority changed from Normal to High
- reviewer Radu Apetrii added
#5 Updated by Radu Apetrii over 1 year ago
- Bullet 1: This most probably means adding another parameter to
SecurityOps$WorkArea.addDomain()which resembles the tenant name. Also, inSecurityOps$Domain, we'll probably need another field,tenantName. - Bullet 2: Now that I'm re-reading bullet 2, I think I made a mistake when writing it. Basically, we give the domain name, and only after that we know which tenant was selected. We don't know in advance the tenant name so that we can check against the domain's tenant name.
- So, assuming I haven't missed anything obvious here, I believe this bullet can be skipped.
- Bullet 3: The SQLs from
SecurityOps.readDomainsneed to be updated to contain thetenantNamecolumn as well.
Depending on what we discover while implementing this feature, or some customer requests that may appear, the notes might suffer some changes. For now, I think bullets 1 and 3 are a good starting point.
#6 Updated by Greg Shah over 1 year ago
What is the 4GL code in OE that would integrate with this feature? I want to make sure that we are not doing something that is incompatible.
#7 Updated by Eric Faulhaber over 1 year ago
- Priority changed from High to Normal
#8 Updated by Ovidiu Maxiniuc about 1 year ago
- Status changed from New to WIP
- % Done changed from 0 to 80
In the meet today, I affirmed that the first and last bullet are already implemented, just need to be tested/confirmed.
However, according to Radu's #8863-5, the second 'bullet can be skipped' as well. I would say that we need to double-check this as well.
Conclusion: this task is about testing features implemented in other (tenant-related) tasks rather than implemented them now.
#9 Updated by Ovidiu Maxiniuc about 1 year ago
- % Done changed from 80 to 100
I double checked all three bullets and all of them are working as expected.
This task can be closed.
#10 Updated by Greg Shah about 1 year ago
- Status changed from WIP to Closed