Why wont this code run

spiceuser-s8au2 · 2020-07-06T17:10:29.000Z

#These two lines will import will give you the access you need to work in Office 365
#Need to know the the email and password for the Portal you are trying to access
Connect-MsolService #Opens up the login for Office 365
import-Module MsOnline #Allows you to run as Admin in Office 365

#connect to Exchange to allow access to emails of client
$exchangeSession = New-PSSession -ConfigurationName Microsoft.Exchange -ConnectionUri " https://outlook.office365.com/powershell-liveid" ; -Credential $Usercredential -Authentication “Basic” -AllowRedirection
Import-PSSession $exchangeSession -DisableNameChecking

#Gives you the list of users to you and idea of what you are working with
#Get-MsolUser
#Get-MsolGroup gives you the groups so you can find what specific group your are looking for and its ObjectId
#Can us Get-MsolGroupMember -GroupObjectId to see who is already in the group
#If you know the group name you can single it out by using this line of code you’ll have to change “Office” for your specific GroupObjectId
Get-MsolGroup -MaxResult 100000|Where-Object{$_.DisplayName -eq “DistGroup”}

#Create a CSV file with just UserPrincipalName from MsolUser
Get-MsolUser|Format-Table -Property UserPrincipalName|Out-File -FilePath C:.CSV
#Need to change file path when necessary
#Use Get-Content to check if you have everthing you need

#Specify DistirbutionGroup
#Change File Path and Distribution to fit needs
#Add-DistributionGroupMember -Identity “DistGroup” -Member “angela@email.com” -BypassSecurityGroupManagerCheck (this allows to put one person in at a time)

Import-CSV C:.CSV | ForEach {Add-DistributionGroupMember -Identity “DistGroup” -Member $_.UserPrincipalName -ByPassSecurityGroupManagerCheck}

!!!THIS IS THE RROR I KEEP GETTING!!!

Cannot validate argument on parameter ‘Member’. The argument is null. Provide a valid value for the argument, and then
try running the command again.

CategoryInfo : InvalidData: ( [Add-DistributionGroupMember], ParameterBindingValidationException
FullyQualifiedErrorId : ParameterArgumentValidationError,Add-DistributionGroupMember
PSComputerName : outlook.office365.com

Neally · 2020-07-06T17:29:21.000Z

If you post code, please use the ‘Insert Code’ button. Please and thank you!

PLEASE READ BEFORE POSTING! Read if you're new to the PowerShell forum! Programming & Development

Hi, and welcome to the PowerShell forum! Don’t apologize for being a “noob” or “newbie” or “n00b.” There’s just no need – nobody will think you’re stupid, and the forums are all about asking questions. Just ask! Use a descriptive subject. Don't say "Need help" or "PowerShell Help", actually summarize what the problem is. It helps the rest of us keep track of which problem is which. Don’t post massive scripts. We’re all volunteers and we don’t have time to read all that, nor will we copy, past…

poons · 2020-07-06T17:30:36.000Z

Are you getting the error on running this bit of your script?

Import-CSV C:\TechResources\PSTest\UserMarucci.CSV | ForEach {Add-DistributionGroupMember -Identity “All Marucci Employees Dist” -Member $_.UserPrincipalName -ByPassSecurityGroupManagerCheck}

If so, are you sure you always have a value in each row for your “UserPrincipalName” column in your UserMarucci.CSV file?

spiceuser-s8au2 · 2020-07-06T17:47:14.000Z

If there are white spaces this wont run?

spiceuser-s8au2 · 2020-07-06T17:49:29.000Z

it goes like this

1 whitespace

2 UserPrincipalName 3---------------------------- 4 starts listing emails

Neally · 2020-07-06T17:49:31.000Z

spiceuser-s8au2:

If there are white spaces this wont run?

white-spaces in a file path can cause issues, yes, if it’s not properly quoted

C:\TechResources\PSTest\UserMarucci.CSV

# VS

"C:\TechResources\PSTest\UserMarucci.CSV"

spiceuser-s8au2 · 2020-07-06T17:51:52.000Z

This is my first time using powershell so im still trying to get use to it.
I added the quotes around the filepath and im still getting the same results

Neally · 2020-07-06T18:05:35.000Z

spiceuser-s8au2:

Import-CSV C:\TechResources\PSTest\UserMarucci.CSV | ForEach {Add-DistributionGroupMember -Identity “All Marucci Employees Dist” -Member $_.UserPrincipalName -ByPassSecurityGroupManagerCheck}

What does your CSV look like?

Do you have a column called ’ UserPrincipalName’ ?

spiceuser-s8au2 · 2020-07-06T18:17:19.000Z

yes i do

marlincompconsult · 2020-07-06T19:21:49.000Z

Delete (completely remove) the row with white space, and the row with dashes.

UserPrincipalName
Username1
Username2
…

adam-h · 2020-07-06T20:09:37.000Z

When I run into an issue like this, I try to break it up into smaller chunks.

For example:

Import-CSV C:\TechResources\PSTest\UserMarucci.CSV | ForEach {Add-DistributionGroupMember -Identity "All Marucci Employees Dist" -Member $_.UserPrincipalName -ByPassSecurityGroupManagerCheck}

Becomes:

$list = import-csv C:\TechResources\PSTest\UserMarucci.CSV
Foreach($i in $list){Add-DistributionGroupMember -Identity "All Marucci Employees Dist" -Member $i.UserPrincipalName -ByPassSecurityGroupManagerCheck}

…or even replace the script block there with a simple {$i.UserPrincipalName} is being passed along successfully.

jimlong3 · 2020-07-07T10:48:44.000Z

marlincomp:

Delete (completely remove) the row with white space, and the row with dashes.

UserPrincipalName
Username1
Username2
…

This.

A .CSV isn’t supposed to be “pretty”, with formatting and visual cues. Just the facts, man.

PS: Style point: Rather than starting each line of a block comment with ‘#’, make a real comment block thusly:

<#
That started the comment block.

Add anything you like in here as a comment.

Now end the comment block with this:
#>

Finally, you could see the answer yourself with this:

Import-CSV "filename.csv" | ForEach {"UPN is $($_.UserPrincipalName)"}

Just substitute something that will dump whatever you’re looking at to the screen instead of the big function you’ll end up with.

HTH…