oslabs-beta · mjhuff · Apr 17, 2023 · Apr 17, 2023 · Apr 17, 2023 · Apr 17, 2023
diff --git a/README.md b/README.md
@@ -6,12 +6,23 @@
 
 [![License](https://img.shields.io/github/license/Ileriayo/markdown-badges?style=for-the-badge)](./LICENSE)
 
-<h2 style="margin: 1rem 0">A free open source tool for securely migrating data between cloud providers.<br/>CloudShift currently supports AWS S3, Microsoft Azure Storage Accounts, and Cloudflare R2.</h2>
+<h2 style="margin: 1rem 0">A free open source tool for securely migrating data between cloud providers.<br/>CloudShift currently supports AWS S3, Microsoft Azure Storage Accounts, and Cloudflare R2.<br/>Built on <a href='https://rclone.org/'>RClone</a>.</h2>
 <h2 style="margin: 1rem 0"><a href="https://www.cloudshift.dev/">cloudshift.dev</a></h2>
 </div>
 <hr/>
 <br/>
 
+## Table of Contents
+
+- [Installation & Getting Started](#installation--getting-started)
+- [Usage & Locating Credentials](#usage--locating-credentials)
+- [What CloudShift Does (and Doesn't)](#what-cloudshift-does-and-doesnt)
+- [Philosophy](#philosophy)
+- [Connect With the Team](#connect-with-the-team)
+- [Want to Contribute?](#want-to-contribute)
+
+ <br/>
+
 ## Installation & Getting Started
 
 CloudShift is a web application available as a container on Docker Hub or for direct download by cloning this repo.
@@ -97,13 +108,17 @@ Couldshift does NOT...
 
 - Address specific use-cases that don't apply to most users.
 
+- Currently(!) support same provider transfers.
+
 If you find that CloudShift doesn't provide you with the granular control necessary for your use-case, or you wish to learn more about Rclone (and how the transfer process works), [visit the Rclone docs](https://rclone.org/).
 
 ## Philosophy
 
 For an in-depth read about our motivation, check out our [Medium article here](https://medium.com/cloudshift/cloudshift-data-migration-simplified-f5c4848d7cb8).
 
-If you feel like CloudShift lacks a feature that fits with our design philosophy, [connect with us](#connect-with-the-team) or [get involved](#want-to-contribute)
+CloudShift seeks to abstract as much of the cloud data migration process as possible for the majority of use-cases. If you require granular control of the transfer process, we recommend using the [Rclone CLI](https://rclone.org/).
+
+If you feel like CloudShift lacks a feature that fits with our design philosophy, [connect with us](#connect-with-the-team) or [get involved](#want-to-contribute).
 
 ## Connect With the Team
 

diff --git a/build/assets/favicon.ico b/build/assets/favicon.ico
diff --git a/build/index.html b/build/index.html
@@ -6,7 +6,9 @@
  <meta http-equiv="X-UA-Compatible" content="IE=edge" />
  <meta name="viewport" content="width=device-width, initial-scale=1.0" />
  <title>CloudShift</title>
- <script defer src="index_bundle.js"></script></head>
+ <link rel="icon" type="image/x-icon" href="./assets/favicon.ico">
+ <script defer src="index_bundle.js"></script>
+ </head>
 
  <body class="bg-gradient-to-br from-cyan-100 to-stone-200 !h-screen !v-screen">
  <style>

diff --git a/build/index_bundle.js b/build/index_bundle.js
diff --git a/server/controllers/assignController.js b/server/controllers/assignController.js
@@ -1,7 +1,6 @@
 const AWS = require("aws-sdk");
 
 const getBucketLoc = async (req, res, next) => {
- //REFACTOR THIS. NOTE THAT YOU CAN'T TRANSFER FROM AMAZON TO AMAZON WITH THE CURRENT LOGIC.
  if (
  req.body.originProvider === "AWS" ||
  req.body.destinationProvider === "AWS"
@@ -63,7 +62,6 @@ const getBucketLoc = async (req, res, next) => {
 
 //This controller parses the req.body and assigns provided input variables to variables that can be used by the fsController.
 const assignVariablesForFS = (req, res, next) => {
- //DO ERROR LOGIC HERE IF ALL THESE VARIABLES DON'T EXIST.
  const {
  originProvider,
  originAccessId,

diff --git a/server/controllers/rcloneController.js b/server/controllers/rcloneController.js
@@ -2,7 +2,6 @@ const rclone = require("rclone.js");
 const { resolve } = require("path");
 const errorGenerator = require("../services/errorGenerator");
 
-//NEED LOTS OF ERROR HANDLING LOGIC HERE.
 const rCloneCopyController = (req, res, next) => {
  // Build the strings for rClone to do the copying.
  const {
@@ -12,6 +11,14 @@ const rCloneCopyController = (req, res, next) => {
  destinationBucket,
  } = req.body;
 
+ //CloudShift currently does not support same origin and destination transfer.
+ if (originProvider === destinationProvider)
+ return next({
+ log: "Error in rcloneListBuckets middleware.",
+ message:
+ "CloudShift currently does not support same origin/desintation provider transfer.",
+ });
+
  const originString = `${originProvider.toLowerCase()}:${originBucket.toLowerCase()}`;
  const destinationString = `${destinationProvider.toLowerCase()}:${destinationBucket.toLowerCase()}`;