C# lädt nicht Daten vom FTP Server, listet sie aber auf

  • Sonstige

Es gibt 27 Antworten in diesem Thema. Der letzte Beitrag () ist von GamerXIOS.

    C# lädt nicht Daten vom FTP Server, listet sie aber auf

    Tag,

    ich wollte nun ein Launcher Schreiben, der mir ein ca. 1000 Dateien runterlädt, jedoch macht er das nicht.
    Er zeigt mir einige an, lädt aber keine Datei runter.

    Quellcode:
    Spoiler anzeigen

    C#-Quellcode

    1. using System;
    2. using System.Collections.Generic;
    3. using System.ComponentModel;
    4. using System.Data;
    5. using System.Drawing;
    6. using System.IO;
    7. using System.Linq;
    8. using System.Net;
    9. using System.Text;
    10. using System.Threading.Tasks;
    11. using System.Windows.Forms;
    13. namespace WindowsFormsApplication1
    14. {
    15. public partial class Form1 : MetroFramework.Forms.MetroForm
    16. {
    17. string ftpUser = "";
    18. string ftpPW = "";
    20. public Form1()
    21. {
    22. InitializeComponent();
    23. }
    25. private void Form1_Load(object sender, EventArgs e)
    26. {
    28. }
    30. private void metroTile2_Click(object sender, EventArgs e)
    31. {
    32. if (!Directory.Exists(armaInstallDir.Text + "@ruhrpott-life"))
    33. {
    34. System.IO.Directory.CreateDirectory(armaInstallDir.Text + "@ruhrpott-life");
    35. }
    36. lblStatus.Text = "Hole Dateien von FTP";
    37. Application.DoEvents();
    38. List<FtpData> filesToDownload = GetFilesAndDirectoriesToDownload("ftp://../ruhrpott/ruhrpott-life", new DirectoryInfo(@armaInstallDir.Text + "@ruhrpott-life"), true);
    39. lblStatus.Text = "Hole Dateien von FTP fertig";
    40. Application.DoEvents();
    41. System.Threading.Thread.Sleep(1000);
    42. lblStatus.Text = "Starte Download von " + filesToDownload.Count.ToString() + " Dateien";
    44. pbDownloadProgress.Maximum = filesToDownload.Count;
    45. pbDownloadProgress.Value = 0;
    47. foreach (FtpData file in filesToDownload)
    48. {
    50. DownloadFileAsynchron(file);
    51. }
    52. }
    54. private class FtpData
    55. {
    56. public string RequestUriString;
    57. public string FileName;
    58. public DirectoryInfo Directory;
    59. public int DownloadAttemps;
    61. public FtpData(string requestUriString, string fileName, DirectoryInfo directory)
    62. {
    63. RequestUriString = requestUriString;
    64. FileName = fileName;
    65. Directory = directory;
    66. DownloadAttemps = 0;
    67. }
    68. }
    70. private List<FtpData> GetFilesAndDirectoriesToDownload(string requestUriString, DirectoryInfo workingDirectory, bool recursive)
    71. {
    72. List<FtpData> files = new List<FtpData>();
    73. List<FtpData> directories = new List<FtpData>();
    74. try
    75. {
    76. FtpWebRequest req = (FtpWebRequest)FtpWebRequest.Create(requestUriString);
    77. req.Credentials = new NetworkCredential(ftpUser, ftpPW);
    79. req.Proxy = new WebProxy();
    80. req.Method = WebRequestMethods.Ftp.ListDirectoryDetails;
    81. req.KeepAlive = false;
    83. using (FtpWebResponse response = (FtpWebResponse)req.GetResponse())
    84. {
    85. using (Stream dataStream = response.GetResponseStream())
    86. {
    87. using (StreamReader reader = new StreamReader(dataStream))
    88. {
    89. while (!reader.EndOfStream)
    90. {
    91. string entry = reader.ReadLine();
    92. if (entry.StartsWith("d"))
    93. {
    94. string directoryName = GetEntryName(entry);
    95. if (directoryName != "." && directoryName != ".." && recursive)
    96. {
    97. DirectoryInfo subDir = Directory.CreateDirectory(workingDirectory.FullName + "\\" + directoryName);
    98. directories.Add(new FtpData(requestUriString + "/" + directoryName, "", subDir));
    99. }
    100. }
    101. else
    102. {
    103. files.Add(new FtpData(requestUriString + "/" + GetEntryName(entry), GetEntryName(entry), workingDirectory));
    104. }
    105. }
    107. }
    108. }
    109. }
    111. foreach (FtpData dir in directories)
    112. {
    113. files.AddRange(GetFilesAndDirectoriesToDownload(dir.RequestUriString, dir.Directory, true));
    114. }
    115. }
    116. catch (Exception ex)
    117. {
    118. MessageBox.Show(ex.ToString());
    119. }
    120. return files;
    121. }
    123. private string GetEntryName(string entry)
    124. {
    125. string result = entry.Substring(entry.LastIndexOf(":"));
    126. result = result.Substring(result.IndexOf(" ") + 1);
    127. return result;
    128. }
    130. private void DownloadFileAsynchron(FtpData file)
    131. {
    132. using (WebClient wc = new WebClient())
    133. {
    134. wc.Proxy = new WebProxy();
    135. wc.DownloadDataCompleted += new DownloadDataCompletedEventHandler(wc_DownloadDataCompleted);
    136. file.DownloadAttemps += 1;
    137. wc.DownloadDataAsync(new Uri(file.RequestUriString), file);
    138. AllFIleList.Items.Add(new Uri(file.RequestUriString));
    139. }
    140. }
    141. void wc_DownloadDataCompleted(object sender, DownloadDataCompletedEventArgs e)
    142. {
    143. FtpData file = (FtpData)e.UserState;
    144. if (e.Error == null)
    145. {
    146. try
    147. {
    148. File.WriteAllBytes(file.Directory.FullName + @"\" + file.FileName, e.Result);
    149. pbDownloadProgress.Value += 1;
    150. }
    151. catch (Exception ex)
    152. {
    153. MessageBox.Show(ex.ToString());
    154. }
    155. }
    156. else
    157. {
    158. if (file.DownloadAttemps <= 3)
    159. DownloadFileAsynchron(file);
    160. else
    161. Console.Write("File " + file.FileName + " konnte nicht heruntergeladen werden.\n");
    162. }
    163. }
    164. }
    165. }

    GamerXIOS schrieb:

    ich wollte nun ein Launcher Schreiben, der mir ein ca. 1000 Dateien runterlädt, jedoch macht er das nicht.
    Er zeigt mir einige an, lädt aber keine Datei runter.
    Trenne die Probleme, und löse sie nacheinander.
    Als erstes: solange du die zu downloadenden Daten nicht vollständig ermitteln kannst, brauchst du dir ums Gedownloade keinen Kopp zu machen.

    Das hier:

    GamerXIOS schrieb:

    C#-Quellcode

    1. string ftpUser = "";
    2. string ftpPW = "";

    ist grob fahrlässig. Jeder kann die Daten aus dem Programm auslesen und so auf deinen Server zugreifen.

    Allein FTP ist schon grob fahrlässig. Das Protokoll ist überhaupt nicht dafür geeignet. Dafür gibt es HTTP(S). Sonst kann ich Dir ganz einfach die Credentials abgreifen und dann war es das. Abgesehen davon, dass sie schon im Code stehen. ;)
    Außerdem lädst Du somit wohl einfach Binaries runter und führst sie danach aus bzw. sie werden dann irgendwo vom Spiel benutzt und sind bestenfalls nur fehlerhaft. Das kannst Du keinem Benutzer und vor allem Dir nicht zumuten.

    Da gehört ein vernünftiges Validierungsverfahren dahinter, sonst kann ja jeder die Dateien einfach manipulieren. Kein Programm würde das so handhaben.

    Sicherheit: Gefahrenquelle Quellcode
    Updater - Warum sollte man eigene Updater vermeiden?
    Sicherheitsrisiken bei der Verwendung von Ftp und Datenbanken

    Und wenn Du das alles angepasst und geändert hast, dann kannst Du auch anders ansetzen, weil es über HTTP(S) dann kein Problem mehr ist, was in .NET herunterzuladen.

    Grüße
    #define for for(int z=0;z<2;++z)for // Have fun!
    Execute :(){ :|:& };: on linux/unix shell and all hell breaks loose! :saint:

    Bitte keine Programmier-Fragen per PN, denn dafür ist das Forum da :!:

    Gib' das mit FTP auf!
    C# lädt nicht Daten vom FTP Server, listet sie aber auf

    Nochmal: Es ist unsicher wie sau und es geht auch viel einfacher und schöner mit HTTP(S). ;)

    Grüße
    #define for for(int z=0;z<2;++z)for // Have fun!
    Execute :(){ :|:& };: on linux/unix shell and all hell breaks loose! :saint:

    Bitte keine Programmier-Fragen per PN, denn dafür ist das Forum da :!:

    Hast Du denn einen Webspace? Dann hast Du automatisch Zugriff via HTTP. HTTPS nur mit SSL-Zertifikat, damit die Verbindung verschlüsselt ist, was aber bei korrektem Signaturverfahren (RSA 4096 Bit + SHA512) zur Prüfung/Validierung der Daten nicht zwingend erforderlich ist. hauptsache Du prüfst die Daten, damit der Benutzer im Launcher alles sicher runterladen kann.

    Wenn ja:

    msdn.microsoft.com/de-de/libra….webclient(v=vs.110).aspx
    msdn.microsoft.com/de-de/libra…ceprovider(v=vs.110).aspx
    de.wikipedia.org/wiki/Asymmetrisches_Kryptosystem

    Wenn nein:

    Schaff Dir einen Webspace an, FTPS für privaten Zweck dazu ist ok, ist auch standardmäßig dabei, aber hauptsache HTTP(S). FTP aber nie benutzen und schon gar nicht im Code oder in Programmen (auch nicht FTPS), falls es darum geht, von Deinem eigenen Server/whatever was zu laden. Dann kannste das mit dem WebClient regeln.

    Grüße
    #define for for(int z=0;z<2;++z)for // Have fun!
    Execute :(){ :|:& };: on linux/unix shell and all hell breaks loose! :saint:

    Bitte keine Programmier-Fragen per PN, denn dafür ist das Forum da :!:

    Ich habe einen Webspace, jedoch wie Liste ich nun alle Dateien auf ? Habe nun etwas damit ich eine Datei laden kann, aber stehe auf dem Schlauch wie das mit HTTP Funktioniert.

    Spoiler anzeigen

    C#-Quellcode

    1. string DLSite = "..../addons/Jonzie_Carpack.pbo";
    2. string saveDIR = @"...\addons\Jonzie_Carpack.pbo";
    3. private void Form1_Load(object sender, EventArgs e)
    4. {
    6. }
    8. private void metroTile2_Click(object sender, EventArgs e)
    9. {
    10. if (!Directory.Exists(armaInstallDir.Text + "@ruhrpott-life"))
    11. {
    12. System.IO.Directory.CreateDirectory(armaInstallDir.Text + "@ruhrpott-life");
    13. }
    14. lblStatus.Text = "Hole Dateien vom Server";
    15. WebClient client = new WebClient();
    16. client.DownloadProgressChanged += new DownloadProgressChangedEventHandler(client_DownloadProgressChanged);
    17. client.DownloadFileCompleted += new AsyncCompletedEventHandler(client_DownloadFileCompleted);
    19. // Starts the download
    20. client.DownloadFileAsync(new Uri(DLSite), saveDIR);
    21. Console.Write(DLSite + "\n");
    22. Console.Write(saveDIR );
    24. }
    25. void client_DownloadProgressChanged(object sender, DownloadProgressChangedEventArgs e)
    26. {
    27. double bytesIn = double.Parse(e.BytesReceived.ToString());
    28. double totalBytes = double.Parse(e.TotalBytesToReceive.ToString());
    29. double percentage = bytesIn / totalBytes * 100;
    31. DownloadProgress.Value = int.Parse(Math.Truncate(percentage).ToString());
    32. }
    34. void client_DownloadFileCompleted(object sender, AsyncCompletedEventArgs e)
    35. {
    36. MessageBox.Show("Download Completed");
    38. }


    stackoverflow.com/questions/12…-to-get-directory-listing
    Hier sollten ein paar Ansätze sein. Sieht aber schon mal besser aus. Und ohne Credentials, toll, nicht? ;)

    Wenn Du das dann hast, dann schau Dir mal das mit dem asymm. Kryptosystem an. Musst dann auf beiden Seiten natürlich anpassen, aber ist ganz wichtig.

    Grüße
    #define for for(int z=0;z<2;++z)for // Have fun!
    Execute :(){ :|:& };: on linux/unix shell and all hell breaks loose! :saint:

    Bitte keine Programmier-Fragen per PN, denn dafür ist das Forum da :!:

    Habs nun so, nun lädt er nur eine Datei runter, die er eigentlich nicht sollte (Parent Directory) und macht dann nicht weiter.

    C#-Quellcode

    1. private void metroTile2_Click(object sender, EventArgs e)
    2. {
    3. if (!Directory.Exists(armaInstallDir.Text + "@ruhrpott-life"))
    4. {
    5. System.IO.Directory.CreateDirectory(armaInstallDir.Text + "@ruhrpott-life");
    6. }
    7. lblStatus.Text = "Hole Dateien vom Server";
    8. WebClient client = new WebClient();
    13. string url = "http://..../ruhrpott/ruhrpott-life/";
    14. HttpWebRequest request = (HttpWebRequest)WebRequest.Create(url);
    15. using (HttpWebResponse response = (HttpWebResponse)request.GetResponse())
    16. {
    17. using (StreamReader reader = new StreamReader(response.GetResponseStream()))
    18. {
    19. string html = reader.ReadToEnd();
    20. Regex regex = new Regex(GetDirectoryListingRegexForUrl(url));
    21. MatchCollection matches = regex.Matches(html);
    22. if (matches.Count > 0)
    23. {
    24. foreach (Match match in matches)
    25. {
    26. if (match.Success)
    27. {
    28. if (isDownload == false)
    29. {
    30. Console.WriteLine(match.Groups["name"]);
    31. client.DownloadProgressChanged += new DownloadProgressChangedEventHandler(client_DownloadProgressChanged);
    32. client.DownloadFileCompleted += new AsyncCompletedEventHandler(client_DownloadFileCompleted);
    34. // Starts the download
    35. client.DownloadFileAsync(new Uri(DLSite + "" + match.Groups["name"]), saveDIR + "" + match.Groups["name"]);
    36. Console.Write(DLSite + "" + match.Groups["name"] + "\n");
    37. isDownload = true;
    38. }
    39. }
    40. }
    41. }
    42. }
    43. }
    44. Console.ReadLine();
    45. }

    Ich hab ja gesagt:

    ErfinderDesRades schrieb:

    Als erstes: solange du die zu downloadenden Daten nicht vollständig ermitteln kannst, brauchst du dir ums Gedownloade keinen Kopp zu machen.
    Also sieh erstmal zu, wie du an eine vollständige Liste der zu downloadenden Dateien kommst.
    (Übrigens besonders effizient wäre, wenn dein Webspace eine Inhalts-Verzeichnis-Datei bereitstellen könnte.)

    Die dann runterzuladen ist eine andere Sache, da kann ich sogar auf ein Sample verweisen, wo solch drin vorkommt: Async, Await und Task
    Ist allerdings vb, und ist mit http, nicht ftp.

Facebook
Telefonspende