Skip when node_modules not found

It seems that finding all tasks in a project fails when the extension finds a gulpfile without a corresponding set of installed node_modules. Instead of failing out completely, could that particular gulpfile just be skipped, perhaps with a notice in the extension's output pane?

Hi @garrettw
Are you using workspaces in vscode?
If so, there is a known issue with the extension resolving gulp multiple times (per workspace project). I started a solution, but haven't got around to completing it. I am hoping to shortly so will let you when it is done.
If not, can you provide some more info around your project structure.
Thanks,
Nick.

I am using workspaces. When I submitted this issue, I was working on a Drupal 7 project where I have a custom module and a custom theme, both of which have their own gulpfiles (rather than one at the root). I was making changes to the theme, and I had already run yarn install in that directory, but I hadn't in the module's directory -- and according to the output pane, the module was where this extension was getting hung up. So I went and ran yarn install in that directory and refreshed the Gulp Tasks pane, and it worked just fine.


Plan to create a separated vte crate?

When I looked for vte wrapper of Rust, I found your SolidOak app uses a forked version of gtk-rs, and it has a vte wrapper. Do you have any plan to separate the implementation of vte wrapper as a crate?

I always wanted to do that, but at the time I wrote it I was not able to break it into a separate crate because it uses macros from gtk-rs, which were not able to cross crate boundaries. I haven't checked recently on whether that has changed; if it has, I would definitely be willing to break it out.

I have understood the reason why it's forked.
I don't read the code in detail, do "macros from gtk-rs" mean "struct_Widget!", "impl_drop!", or "impl_TraitWidget!" ? These macros seem to be disappeared in the current gtk-rs/gtk repository. The style of code for widgets seem to be completely changed (I see: old:gtk/src/widgets/text_view.rs , new:gtk/src/auto/text_view.rs). It may be worth to check the new implementation.

Any news on this?

Better late than never :D I am now using a separate create for vte:
https://github.com/oakes/vte-rs


Looking for Maintainers

Hey all :),
As you may have noticed, recently, I've not put the attention Restangular deserves. Lots of people have been using Restangular and despite it's at a stable point, there're still a few bugs / features that I'd like to see implemented. For example, making it compatible with newer Lodash version.
However, recently I've changed roles and unfortunately I don't have the time to mantain Restangular as I used to, and I can't give Restangular the time it really deserves.
This is why @ConcurrentHashMap will now help solving some issues and merging PRs. However if somebody else wants to join him in helping work with PRs / new features, all the help is welcome to continue making this project awesome :).
Thanks all for the love <3

Hi @mgonto, first good luck in your new position, second, happy to help @ConcurrentHashMap as a maintainer in the project. If you could add me there.
Cheers,

Thanks @pedromarce :).

So, thank you for your trust @mgonto. I will do my best to take care of PRs and issues.
@pedromarce: Would you like to contact me via mail to organize us a little bit? :)

Sure @ConcurrentHashMap, could we use twitter (mine is @pedromarce)?

@mgonto thanks for all your hard work
@ConcurrentHashMap thanks for coming on! I'd like to help as best I can. I'm unsure as to how much time I'll be able to dedicate to helping, but I'm quite familiar with Restangular and would love to help keep  making it awesome.

:+1: Sounds great!

@ConcurrentHashMap Is there a gitter/Slack channel or plans for one?

Good idea @daviesgeek, I have created restangular.slack.com, I will send invitations for yourself and @ConcurrentHashMap.

I've been using Restangular for a little over a year, and more recently have actually been digging around and making some changes/enhancements in my own fork.  So, I've become familiar enough that I'd be comfortable contributing where I can.  If you guys still want/need the help, shoot me an invite to the Slack channel!

Hi there, sounds good to me, could you tell me where to send the invite? Cheers,

<EMAIL_ADDRESS>

Does this mean there is no chance of seeing a restangular 2.0 anytime soon?

@pedromarce @ConcurrentHashMap @daviesgeek I've been using restangular for over two years and I've also done some changes to it in my own fork. I couldn't say that I've got the expertise to continue on features, but I see that this github project would need some love. There's a lot of issues and PRs hanging around that would need to be resolved. If you need help with that, I'd be happy to contribute with a little of my time.

@bostrom if you could help out with cleaning up some of the issues and PRs, that would be awesome. I just don't have the time right now :/

@daviesgeek sure! I can spend some time on it now and then, perhaps this repo will get back in shape sooner or later. Are you actively discussing the state of things somewhere (slack/gitter)?

@bostrom Yes we have a Slack channel. What's your email? I'll add you

@daviesgeek fbostrom at gmail.

@bostrom Thanks! Invite just sent out to ya!

Hi. everyone.
Is anyone used Restangular to post image or files to Laravel backend?


AOT Fails with Angular2 external module : Error encountered resolving symbol values statically.

From @hireton on June 9, 2017 10:34
Ionic version:
[ ] 1.x
[ ] 2.x
[x] 3.10.2
I'm submitting a ...
[x] bug report
[ ] feature request
[ ] support request
Current behavior:
Building with --prod option fails if project uses an external module with Angular2 dependencies (@ angular/core)
I tried moving deps to dependencies or devDependencies with no change.
Error encountered resolving symbol values statically. Calling function 'makeDecorator', function calls are not supported.
Build succeeds without --prod option or if the external module has no Angular2 dependencies.
Expected behavior:
Build with --prod option should succeed.
Steps to reproduce:
1. Create new project
ionic start testproject blank

2. Create external plugin
mkdir testplugin
cd testplugin

add package.json TestClass.ts
4. Install plugin
Install and link (or copy plugin to node_modules)
npm install
npm link
cd ../testproject
npm link @myexternals/testplugin

5. Register plugin in app.module.ts
6. Build in prod mode
ionic cordova build android  --prod

Related code:
3 files updated from ionic bootstrap :
testplugin/package.json
{
  "name": "@myexternals/testplugin",
  "scripts": {
    "tsc": "tsc",
    "tsc:w": "tsc -w"
  },
  "dependencies": {
  },
  "repository": {},
  "license": "",
  "peerDependencies": {
    "@angular/core": ">=2.0.0"
  },
  "devDependencies": {
    "@angular/core": "4.1.2",
    "rxjs": "5.1.1",
    "typescript": "^2.0.10",
    "@types/node": "^6.0.46"
  },
  "version": "0.1.1"
}

testplugin/TestClass.ts
import {Component} from "@angular/core"
@Component({
    selector:"testclass",
    template:"

"
})
export class  TestClass{
    constructor(){
    }
}

testproject/src/app/app.module.ts
import { BrowserModule } from '@angular/platform-browser';
import { ErrorHandler, NgModule } from '@angular/core';
import { IonicApp, IonicErrorHandler, IonicModule } from 'ionic-angular';
import { SplashScreen } from '@ionic-native/splash-screen';
import { StatusBar } from '@ionic-native/status-bar';
import { MyApp } from './app.component';
import { HomePage } from '../pages/home/home';

import {TestClass} from "@myexternals/testplugin/TestClass"

@NgModule({
  declarations: [
    MyApp,
    HomePage,
    TestClass
  ],
  imports: [
    BrowserModule,
    IonicModule.forRoot(MyApp)
  ],
  bootstrap: [IonicApp],
  entryComponents: [
    MyApp,
    HomePage,
    TestClass
  ],
  providers: [
    StatusBar,
    SplashScreen,
    {provide: ErrorHandler, useClass: IonicErrorHandler}
  ]
})
export class AppModule {}

Files in this plunker : http://plnkr.co/edit/GcygCJ6GKuwdFpipZuEA?p=preview
Other information:
Build error :
Error: Error encountered resolving symbol values statically. Calling function 'makeDecorator', function calls are not supported. Consider replacing the function or lambda with a reference to an exported function, resolving symbol Injectable in D:/test/testproject/node_modules/<EMAIL_ADDRESS>resolving symbol ɵf in D:/test/testproject/node_modules/<EMAIL_ADDRESS>resolving symbolɵ f in D:/test/testproject/node_modules/<EMAIL_ADDRESS>
Ionic info:
Node       : v6.10.3
OS         : Windows 10
Xcode      : not installed
ios-deploy : not installed
ios-sim    : not installed
Copied from original issue: ionic-team/ionic#11993

Hello! Thanks for opening an issue with us! As this seems like more of a support question I will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!

@jgw96 I was hoping to see a response to this issue since it has a pretty clear cut example to reproduce and I am also experiencing this error when using an external module to share components among multiple apps.  We are currently having to do production builds via --minifyjs and skipping --aot.


Implement transform data for tarams schema

convert_status = fn status, data ->
    text =
      case status do
        0 -> "draft"
        1 -> "published"
        2 -> "banned"
      end

    text = if data.deleted, do: "deleted", else: text

    {:ok, text}
  end

  schema = %{
    status: [:integer, as: :product_status, into: convert_status]
  }

  data = %{
    status: 0,
    deleted: true
  }

Pull Request Test Coverage Report for Build ea68247a2fe5aac3e8c7fe66a32e6793f484606c-PR-12

21 of 21 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 91.852%

Totals

Change from base Build a06467bb14e096e509a1c4a808ada97ecfa3e02a:
0.4%

Covered Lines:
124

Relevant Lines:
135

💛  - Coveralls


TEST PR - DO NOT MERGE

Can one of the admins verify this patch?

add to whitelist

add to whitelist


Add support for security vendor extensions

Description
Adds support for rendering/handling vendor extensions in securitySchemes.
Motivation and Context
Addresses an issue with our Code Security API.
How Has This Been Tested?
See deploy preview.
Types of changes

Bug fix (non-breaking change which fixes an issue)

I'm curious what extensions this fix is for, if you wouldn't mind sharing?

Hi @jasikpark, the extensions will vary by vendor, but they will all begin with x-. Basically, the plugin will now render/include whatever extensions are passed under security schemes.

ah, ig I was hoping to see an example of what that would look like, but it sounds like extensions are mostly invented and used privately for different apis?


RequireAndVerifyClientCert error

Greetings!
I'm trying to configure TLCP to work like standard TLS, with just one key and one certificate for each side. It happens that I get an error message referring to cipher suites: "server: handshake failed: tlcp: no cipher suite supported by both client and server".
Can anyone point me out what's wrong with my code please?
package main

import (
	"bufio"
	"bytes"
	"crypto/rand"
	"encoding/pem"
	"errors"
	"flag"
	"fmt"
	"log"
	"net"
	"os"

	"gitee.com/Trisia/gotlcp/tlcp"
	"github.com/emmansun/gmsm/smx509"
)
var (
	cert      = flag.String("cert", "Certificate.pem", "Certificate path.")
	tcpip     = flag.String("tcp", "", "Encrypted TCP Transfer Protocol. [server|client]")
	key       = flag.String("key", "", "`Private key.")
	iport     = flag.String("ipport", "", "Local Port/remote's side Public IP:Port.")
)

func handleConnection(c net.Conn) {
	log.Printf("Client(TLS) %v connected via secure channel.", c.RemoteAddr())
}

func main() {
	flag.Parse()
	if (*tcpip == "server" || *tcpip == "client") {
		var certPEM []byte
		var privPEM []byte

		file, err := os.Open(*key)
		if err != nil {
			log.Fatal(err)
		}
		info, err := file.Stat()
		if err != nil {
			log.Fatal(err)
		}
		buf := make([]byte, info.Size())
		file.Read(buf)

		var block *pem.Block
		block, _ = pem.Decode(buf)

		if block == nil {
			errors.New("no valid private key found")
		}

		privPEM = buf

		file, err = os.Open(*cert)
		if err != nil {
			log.Fatal(err)
		}
		info, err = file.Stat()
		if err != nil {
			log.Fatal(err)
		}
		buf = make([]byte, info.Size())
		file.Read(buf)
		certPEM = buf

		if *tcpip == "server" {
			var cert tlcp.Certificate
 			cert, err = tlcp.X509KeyPair(certPEM, privPEM)
			cfg := tlcp.Config{Certificates: []tlcp.Certificate{cert, cert}, CipherSuites: []uint16{tlcp.ECC_SM4_GCM_SM3}, ClientAuth: tlcp.RequireAndVerifyClientCert}
			cfg.Rand = rand.Reader

			port := "8081"
			if *iport != "" {
				port = *iport
			}

			ln, err := tlcp.Listen("tcp", ":"+port, &cfg)
			if err != nil {
				log.Fatal(err)
			}

			fmt.Fprintln(os.Stderr, "Server(TLCP) up and listening on port "+port)

			conn, err := ln.Accept()
			if err != nil {
				log.Fatal(err)
			}
			defer ln.Close()

			tlcpcon := conn.(*tlcp.Conn)
			err = tlcpcon.Handshake()
			if err != nil {
				log.Fatalf("server: handshake failed: %s", err)
			} else {
				log.Print("server: conn: Handshake completed")
			}

			state := tlcpcon.ConnectionState()

			for _, v := range state.PeerCertificates {
				derBytes, err := smx509.MarshalPKIXPublicKey(v.PublicKey)
				if err != nil {
					log.Fatal(err)
				}
				pubPEM := pem.EncodeToMemory(&pem.Block{Type: "PUBLIC KEY", Bytes: derBytes})
				fmt.Printf("%s\n", pubPEM)
			}

			go handleConnection(conn)
			fmt.Println("Connection accepted")

			for {
				message, err := bufio.NewReader(conn).ReadString('\n')
				if err != nil {
					fmt.Println(err)
					os.Exit(3)
				}
				fmt.Print("Client response: " + string(message))

				reader := bufio.NewReader(os.Stdin)
				fmt.Print("Text to be sent: ")
				text, err := reader.ReadString('\n')
				if err != nil {
					fmt.Println(err)
					os.Exit(3)
				}
				fmt.Fprintf(conn, text+"\n")
			}
		}

		if *tcpip == "client" {
			var cert tlcp.Certificate
			cert, err = tlcp.X509KeyPair(certPEM, privPEM)
			cfg := tlcp.Config{Certificates: []tlcp.Certificate{cert}, InsecureSkipVerify: true, CipherSuites: []uint16{tlcp.ECC_SM4_GCM_SM3}}

			ipport := "<IP_ADDRESS>:8081"
			if *iport != "" {
				ipport = *iport
			}

			conn, err := tlcp.Dial("tcp", ipport, &cfg)
			if err != nil {
				log.Fatal(err)
			}
			certs := conn.ConnectionState().PeerCertificates
			for _, cert := range certs {
				fmt.Printf("Issuer: \n\t%s\n", cert.Issuer)
				fmt.Printf("Subject: \n\t%s\n", cert.Subject)
				fmt.Printf("Expiry: %s \n", cert.NotAfter.Format("Monday, 02-Jan-06 15:04:05 MST"))
			}
			if err != nil {
				log.Fatal(err)
			}
			defer conn.Close()

			var b bytes.Buffer
			for _, cert := range conn.ConnectionState().PeerCertificates {
				err := pem.Encode(&b, &pem.Block{
					Type: "CERTIFICATE",
					Bytes: cert.Raw,
			        })
				if err != nil {
					log.Fatal(err)
				}
			}
			fmt.Println(b.String())

			for {
				reader := bufio.NewReader(os.Stdin)
				fmt.Print("Text to be sent: ")
				text, err := reader.ReadString('\n')
				if err != nil {
					fmt.Println(err)
					os.Exit(3)
				}
				fmt.Fprintf(conn, text+"\n")

				message, err := bufio.NewReader(conn).ReadString('\n')
				if err != nil {
					fmt.Println(err)
					os.Exit(3)
				}
				fmt.Print("Server response: " + message)
			}
		}
		os.Exit(0)
	}
}

I don't understand what's wrong, if anyone can help me that would be great.
Thanks in advance.

After some tests I realized that it's working, but only with the example certificates, it doesn't work with the certificates generated by my tool EDGETk although it also uses the gmsm library. I can't generate TLCP compliant certificates using my program, and I can't parse the private keys in the library examples either.

To support SM2 private key with RFC 5915 / SEC1 format, i will enhance gotlcp.
To verify client certificate, pls make sure ClientCAs are ready, your server system will NOT support SM2. BTW, please make sure you used correct client side certificates (KeyUsage):

		keyUsages := []x509.ExtKeyUsage{x509.ExtKeyUsageClientAuth, x509.ExtKeyUsageServerAuth}
		if c.config.ClientAuth == RequireAndVerifyAnyKeyUsageClientCert {
			keyUsages = []x509.ExtKeyUsage{x509.ExtKeyUsageAny}
		}

BTW, suggest you to test the case InsecureSkipVerify = false (InsecureSkipVerify 's default value is false), to verify server certificates first.

Very thanks for the tips and I was doing a few things wrong besides that:

I was using a non-self-signed certificate as ClientCA.
I was generating certificates whose Extended Key Usage was Client Authentication only.

I think an extra flag is needed in my program to define the ROOT_CERT_PEM. But I don't understand if it's imperative that the server uses two pairs of keys. From what I could understand it would be less secure to use the same key to sign and encrypt, but I don't understand if this is a rule. What are the implications of using just one key/certificate pair on the server side?
I mean, otherwise I would need to make more changes to flags like secondary key and secondary certificate. Is it possible to use only one pair of keys or is this contraindicated?

@pedroalbanese , in general, tlcp server will use two pairs of keys: one for sign , another for encryption, they are different keys and generated by same CA, that's also the main reason why tlcp is incompatible with tls. If tlcp client supports ECDHE_SM4_CBC_SM3/ECDHE_SM4_GCM_SM3 suites, it also needs two pairs of keys.
In fact, the real certificates request procedure like below:

applicant generates one key pair, and one CSR, sends CSR to CA;
CA generates the sign certificate according the CSR, and also generates one key pair, generates one new CSR (use the requestor's CSR as template source), and then generates the encryption certificate.
CA returns sign certificate, encryption certificate and encryption private key to applicant. (so, CA keeps your encryption private key)

"What are the implications of using just one key/certificate pair on the server side?", for demo and self-test, it's ok, but you will fail to integrate with others,

That's cool! Quite interesting, I will think about how to deal with the flags, for example keeping a single flag and passing two values separated by semicolons. Thank you so much again for your patience and all the information. It is an honor to be able to learn about China's industrial standard.

Hello again (I didn't want to open a new issue);
I'm having problems with the integration with GmSSL. When I try one-way authentication, it works. But when I try two-way authentication, I get the server error (EDGETk) "server: handshake failed: remote error: tlcp: unsupported certificate", about the client certificate (GmSSL).
I am generating the certificate like this:
gmssl certgen -key sm2priv.pem -pass secret -CN Applicant -key_usage digitalSignature -key_usage nonRepudiation -key_usage keyEncipherment -days 365 -ext_key_usage clientAuth -ext_key_usage serverAuth -out sm2cert.pem
But it results in "unsupported certificate" on the server side.
-----BEGIN CERTIFICATE-----
MIIBUzCB+qADAgECAg0ApKf1vYJPqL+vz9xiMAoGCCqBHM9VAYN1MBQxEjAQBgNV
BAMTCUFwcGxpY2FudDAeFw0yMzEwMjYxNTQ2NTRaFw0yNDEwMjUxNTQ2NTRaMBQx
EjAQBgNVBAMTCUFwcGxpY2FudDBZMBMGByqGSM49AgEGCCqBHM9VAYItA0IABAcO
p6tMBTQSpKEzpFl7uloSvDaQ+6As2DTUEfkZs4f/tLZl7Tw1klafGxcTr49UEByy
nVuv1QqnQZk63kop48GjMTAvMA4GA1UdDwEB/wQEAwIF4DAdBgNVHSUEFjAUBggr
BgEFBQcDAgYIKwYBBQUHAwEwCgYIKoEcz1UBg3UDSAAwRQIgehH1sVhp3Y6qUamJ
IxCZXnPGukw5nhIap43vbredqw0CIQD6unH89Cgx6ZfjFxbjbGahSyUSLbVS4YcB
hAY8uq+LJQ==
-----END CERTIFICATE-----
What I'm doing wrong?

What's the server side detail log and what's your client config?

I was getting this error yesterday, I don't know why, but when I tried to reproduce it today to print the error, the protocol worked. I was doing something wrong yesterday and it mysteriously worked today. Finally, EDGETk is now fully compatible with GmSSL!

Thank you for your patience and genius libraries! Thank you and Trisia.


Frontend: Implement location button to get users location and pinpoint it to the map

Also make sure to check that this works with mobile.

Done


bump up libraries to support scala 2.12

remove to while-loop optimization from two cycles to remove legacy dependency
add the newest versions of all libraries which still support all Scala 2.10, 2.11, 2.12

If you really find the two for-loops really must be optimized we can rewrite them to while-loops manually.

Resolves issue #108.

Well, it seems that there is no dl4j version 0.9.2-SNAPSHOT... I can not do anything about it. Changing the default version in the build.sbt did not make much sense. I do not know whether you stick to rolling out appropriate version together or what is your policy on this issue.

@letalvoj you would have to build from source for that

Works for me (actually I was lazy and locally used 0.9.1) but it does not fix the build on Travis. :-(

I found crazy weird optimizing a loop but still using exception handling to break out of it. Thus I removed the scala.util.control.Breaks and used plain old procedural boolean flag.
When you fix the Complex... stuff and release the appropriate version of nd4j-native the tests should be passing.
Fixes also bug https://github.com/deeplearning4j/nd4s/issues/120

@letalvoj

Done ✔

LGTM, so let's merge this so others start can testing it more easily. Thanks!


[Event Request] table 246 "Requisition Line" - OnValidateQuantityOnAfterSetRemainigQty

Can we please get the following event?
field(8; Quantity; Decimal)
{
    Caption = 'Quantity';
    DecimalPlaces = 0 : 5;

    trigger OnValidate()
    begin
        Quantity := UOMMgt.RoundAndValidateQty(Quantity, "Qty. Rounding Precision", FieldCaption(Quantity));

        "Quantity (Base)" := UOMMgt.CalcBaseQty(
            "No.", "Variant Code", "Unit of Measure Code", Quantity, "Qty. per Unit of Measure",
            "Qty. Rounding Precision (Base)", FieldCaption("Qty. Rounding Precision"), FieldCaption(Quantity),
            FieldCaption("Quantity (Base)")
        );

        if Type = Type::Item then begin
            GetDirectCost(FieldNo(Quantity));
            "Remaining Quantity" := Quantity - "Finished Quantity";
            "Remaining Qty. (Base)" := "Remaining Quantity" * "Qty. per Unit of Measure";
            OnValidateQuantityOnAfterSetRemainigQty(Rec, xRec, CurrFieldNo, CurrentFieldNo); //NEW
            if (CurrFieldNo = FieldNo(Quantity)) or (CurrentFieldNo = FieldNo(Quantity)) then
                SetActionMessage;

          [...]

        end;
    end;
}

[IntegrationEvent(false, false)]
local procedure OnValidateQuantityOnAfterSetRemainigQty(var RequisitionLine: Record "Requisition Line"; xRequisitionLine: Record "Requisition Line"; CallingFieldNo: Integer; GlobalCurrentFieldNo: Integer)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


2.8 + CLI: "visit.py" in current path will be imported over the actual visit module

Eric has a script file from ~2006 called "visit.py". When testing 2.8, he ran the cli from the directory containing this file and this file was imported instead of the actual VisIt module. This must be due to Cyrus' changes which bring the CLI closer to an off the shelf python interpreter. Not sure if this is good or bad behavior, we should consider adding sanity checks to the CLI, or directly import the visit module.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1961
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: 2.8 + CLI: "visit.py" in current path will be imported over the actual visit module
Assigned to: -
Category: -
Target version: -
Author: Cyrus Harrison
Start: 08/27/2014
Due date:
% Done: 0%
Estimated time:
Created: 08/27/2014 03:29 pm
Updated: 09/09/2014 01:41 pm
Likelihood: 2 - Rare
Severity: 3 - Major Irritation
Found in version: trunk
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
Eric has a script file from ~2006 called "visit.py". When testing 2.8, he ran the cli from the directory containing this file and this file was imported instead of the actual VisIt module. This must be due to Cyrus' changes which bring the CLI closer to an off the shelf python interpreter. Not sure if this is good or bad behavior, we should consider adding sanity checks to the CLI, or directly import the visit module.
Comments:

@cyrush Is this still relevant?

This can closed. In cli.C a warning message is provided about the potential conflict:
`//
// If there is a file named "visit.py" in the current working directory,
// this will be imported instead of the actual visit python module.
// This is the expected  python interpreter behavior, but it is very
// confusing when users stumble upon this.
//
// We decided to provide an error message to let users identify this case.
//
    FileFunctions::VisItStat_t vstat_info;
    if(FileFunctions::VisItStat("visit.py", &vstat_info) != -1)
    {
        std::ostringstream oss;
        oss <<"!!!! - WARNING - !!!!"
            << std::endl
            <<"You have a file named 'visit.py' in your current working "
            <<"directory. Python's standard module import logic will use "
            <<"this 'visit.py' file instead of the VisIt python module "
            <<"that implements VisIt's python client interface."
            << std::endl
            <<"This will most likely disable all VisIt python client "
            <<"features!"
            << std::endl
            <<"To avoid this please run in a directory without a file "
            <<"named 'visit.py'."
            << std::endl;

        std::cout << oss.str() << std::endl;
        debug1    << oss.str() << std::endl;
    }`

@griffin28 thanks for finding the exact details


Add enabledOperators parameter for workspace name unique check

Description
Add enabledOperators parameter for workspace name unique check. The default value for this flag is ALL means all operators are supported.
For example we have workspace named test-workspace and now we create a new workspace named test test-workspace that will match existing test-workspace by using simple query string and leads name unique check failed.
GET .kibana/_search
{
  "query": {
    "bool": {
      "must": [
        {
          "term": {
            "type": {
              "value": "workspace"
            }
          }
        },
        {
          "simple_query_string": {
            "query": "test test-workspace", // this means match name by test or test-workspace
            "fields": [
              "workspace.name"
            ],
            "flags": "ALL"
          }
        }
      ]
    }
  }
}
# test-workspace will return

Issues Resolved

Screenshot

Testing the changes

Check List

[ ] All tests pass

[ ] yarn test:jest
[ ] yarn test:jest_integration

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] Update CHANGELOG.md
[ ] Commits are signed per the DCO using --signoff

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.05%. Comparing base (d66c8fc) to head (a13aeb2).

Additional details and impacted files
@@                   Coverage Diff                   @@
##           workspace-pr-integr     #306      +/-   ##
=======================================================
- Coverage                67.05%   67.05%   -0.01%
=======================================================
  Files                     3345     3345
  Lines                    65045    65046       +1
  Branches                 10505    10506       +1
=======================================================
- Hits                     43617    43616       -1
- Misses                   18849    18850       +1
- Partials                  2579     2580       +1

Flag
Coverage Δ

Linux_
?

_1
32.59% <0.00%> (-0.01%)
:arrow_down:

_2
55.39% <100.00%> (+<0.01%)
:arrow_up:

_3
43.45% <0.00%> (-0.01%)
:arrow_down:

_4
35.14% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Overall looks good to me. But can we add a integration_test for this case? You can leverage the workspace CRUD integration test file to add the enabledOperators related test cases.

Overall looks good to me. But can we add a integration_test for this case? You can leverage the workspace CRUD integration test file to add the enabledOperators related test cases.

sure, will add a test for it


add DU and IIC, UDSC

Hopefully this should add IIC, UDSC and Delhi University(https://en.wikipedia.org/wiki/University_of_Delhi) to list of colleges. :slightly_smiling_face:

Hey @xandfury : We persist the institutes in a database.
We created this csv file to populate country field for users when we were just introducing the country field. Please check on the website if any institute needs to be added and let me know. Will add it to the db.
Thank you for your contribution :)

In that case, could you please add them? I feel creating an issue for this would be an overkill.

done 👍


Fix babbage-test and conway-test versions

They were not updated when some dependency version bounds were bumped.
Description

Checklist

[x] Commit sequence broadly makes sense and commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[x] When applicable, versions are updated in .cabal and CHANGELOG.md files according to the
versioning process.
[ ] The version bounds in .cabal files for all affected packages are updated. If you change the bounds in a cabal file, that package itself must have a version increase. (See RELEASING.md)
[ ] All visible changes are prepended to the latest section of a CHANGELOG.md for the affected packages. New section is never added with the code changes. (See RELEASING.md)
[x] Code is formatted with fourmolu (use scripts/fourmolize.sh)
[x] Cabal files are formatted (use scripts/cabal-format.sh)
[x] hie.yaml has been updated (use scripts/gen-hie.sh)
[x] Self-reviewed the diff

I guess we have to release these two to CHaP as well, no?

Indeed! Sam already opened a PR.


'A type with the name {FQN} does not exist at...'

Exact code I am having the issue with is available here: https://github.com/bryanloeper/Fallout4Checklist/commit/3d0cd8d0e3c43a70a5a1a97d028be24492a14e74
Key classes:
Fallout4Checklist > FO4 (Portable) > Entities > Locations > Location.cs
Fallout4Checklist > FO4.Web > Data > Fallout4Context.cs

Visual Studio / Extension Version Info
Microsoft Visual Studio Enterprise 2015
Version 14.0.25425.01 Update 3
Microsoft .NET Framework
Version 4.6.01586
Installed Version: Enterprise
Architecture and Modeling Tools   00322-80000-00000-AA436
Microsoft Architecture and Modeling Tools
UML® and Unified Modeling Language™ are trademarks or registered trademarks of the Object Management Group, Inc. in the United States and other countries.
Microsoft Visual Studio Tools for Applications 2015   00322-80000-00000-AA436
Microsoft Visual Studio Tools for Applications 2015
Visual Basic 2015   00322-80000-00000-AA436
Microsoft Visual Basic 2015
Visual C# 2015   00322-80000-00000-AA436
Microsoft Visual C# 2015
Visual C++ 2015   00322-80000-00000-AA436
Microsoft Visual C++ 2015
Windows Phone SDK 8.0 - ENU   00322-80000-00000-AA436
Windows Phone SDK 8.0 - ENU
.ignore   1.2.67
A language service that makes it painless to handle all types of .ignore files such as .gitignore, .tfignore, etc.
Add New File   3.5
The fastest and easiest way to add new files to any project - including files that start with a dot
ASP.NET and Web Tools 2015.1   14.1.20810.0
ASP.NET and Web Tools 2015.1
ASP.NET Web Frameworks and Tools 2012.2   4.1.41102.0
For additional information, visit http://go.microsoft.com/fwlink/?LinkID=309563
ASP.NET Web Frameworks and Tools 2013   5.2.40314.0
For additional information, visit http://www.asp.net/
Azure App Service Tools v2.9.1   14.0.20622.0
Azure App Service Tools v2.9.1
Azure Data Lake Node   1.0
This package contains the Data Lake integration nodes for Server Explorer.
Azure Data Lake Tools for Visual Studio   2.0.6000.0
Microsoft Azure Data Lake Tools for Visual Studio
Bundler & Minifier   2.1.279
Adds support for bundling and minifying JavaScript, CSS and HTML files in any project.
CodeMaid   10.1.93
CodeMaid is an open source Visual Studio extension to cleanup and simplify our C#, C++, F#, VB, PHP, PowerShell, R, JSON, XAML, XML, ASP, HTML, CSS, LESS, SCSS, JavaScript and TypeScript coding.
Common Azure Tools   1.8
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
File Nesting   2.5.62
Automatically nest files based on file name and enables developers to nest and unnest any file manually
GitHub.VisualStudio   1.0
A Visual Studio Extension that brings the GitHub Flow into Visual Studio.
Image Optimizer   3.6.93
Uses industry standard tools to optimize any JPEG, PNG and Gifs - including animated Gifs. Can do both lossy and lossless optimization.
Image Sprites   1.3.38
Boost your website's performance by creating image sprites to reduce the amount of HTTP requests needed.
JavaScript Language Service   2.0
JavaScript Language Service
JavaScript Project System   2.0
JavaScript Project System
Markdown Editor   1.7.148
A full featured Markdown editor with live preview and syntax highligting. Supports GitHub flavored Markdown.
Microsoft .NET Core Tools (Preview 2)   14.1.20810.0
Microsoft .NET Core Tools (Preview 2)
Microsoft Azure HDInsight HQL Service   2.0.6000.0
Language service for Hive query
Microsoft Azure HDInsight Tools for Visual Studio   2.0.6000.0
An integrated development environment for HDInsight application development.
Microsoft Azure Mobile Services Tools   1.4
Microsoft Azure Mobile Services Tools
Microsoft Azure Tools   2.9
Microsoft Azure Tools for Microsoft Visual Studio 2015 - v2.9.40701.9
NuGet Package Manager   3.5.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit http://docs.nuget.org/.
Open Command Line   2.1.175
Opens a command line at the root of the project. Support for all consoles such as CMD, PowerShell, Bash etc. Provides syntax highlighting, Intellisense and execution of .cmd and .bat files.
Package Installer   1.8.85
Makes it easier, faster and more convenient than ever to install Bower, npm, JSPM, TSD, Typings and NuGet packages to any project
PreEmptive Analytics Visualizer   1.2
Microsoft Visual Studio extension to visualize aggregated summaries from the PreEmptive Analytics product.
SQL Server Analysis Services   13.0.1605.88
Microsoft SQL Server Analysis Services Designer
Version 13.0.1605.88
SQL Server Compact & SQLite Toolbox   4.6.0
SQL Server Compact & SQLite Toolbox adds scripting, import, export, rename, query execution and much more to SQL Server Compact & SQLite Data Connections.
SQL Server Data Tools   14.0.60629.0
Microsoft SQL Server Data Tools
SQL Server Integration Services
Microsoft SQL Server Integration Services Designer
Version 13.0.1601.5
SQL Server Reporting Services   13.0.1605.88
Microsoft SQL Server Reporting Services Designers
Version 13.0.1605.88
ToolWindowHostedEditor   1.0
Hosting json editor into a tool window
TypeScript   <IP_ADDRESS>
TypeScript tools for Visual Studio
Visual Studio Spell Check Everywhere   VSSpellCheckEverywhere
An extension that enables spell checking within any Visual Studio file editor or tool window that uses WPF text boxes.
https://GitHub.com/EWSoftware/VSSpellChecker
Visual Studio Spell Checker   VSSpellChecker
An editor extension that checks the spelling of comments, strings, and plain text as you type or interactively with tool windows.
https://GitHub.com/EWSoftware/VSSpellChecker
Visual Studio Tools for Universal Windows Apps   14.0.25527.01
The Visual Studio Tools for Universal Windows apps allow you to build a single universal app experience that can reach every device running Windows 10: phone, tablet, PC, and more. It includes the Microsoft Windows 10 Software Development Kit.
Web Accessibility Checker   1.3.32
The easiest way to perform accessibility checks on any ASP.NET web application. Fully customizable and support all the major international accessibility standards.
Web Compiler   1.11.319
Compiler for LESS, Sass and CoffeeScript files
Web Essentials 2015.3   3.0.235
Adds many useful features to Visual Studio for web developers. Requires Visual Studio 2015
Xamarin   <IP_ADDRESS> (fcbe082)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.
Xamarin.Android   <IP_ADDRESS> (1cf254d)
Visual Studio extension to enable development for Xamarin.Android.
Xamarin.iOS   <IP_ADDRESS> (f37444a)
Visual Studio extension to enable development for Xamarin.iOS.

@bryanloeper There was an issue with scaffolding when the model was in a dependency.
See #251 for more details.
The issue is fixed in the dev branch.
But for a workaround, you can move the model class to the web project for scaffolding and then move it back.

@prafullbhosale Thanks - that worked! Any ETA on when the dev branch will make it to a release? I was thinking it might be a different root cause since Tooling's Preview2 just got updated a few days ago.

@bryanloeper Sorry for the delay in response here.
The fix is now available on nuget in version 1.0.0-preview2-update1


react-real-time question

Category

[ x] Question
[ ] Bug
[ ] Enhancement

In the use case where you have multiple newsfeed lists on different sites, is there a way to pass not only the itemId, but also the listId of the item, and site url in a logic app? would a custom connector do this? or are we currently stuck creating a webhook for this?
Thanks!

Hi @davidhartman. Unfortunately, even you can retrieve custom properties from a SharePoint list item, properties like listId or siteUrl are not available in the output of the Logic App action. Maybe webhooks are your best solution in this case.

@FranckyC thanks for the reply! makes total sense!


When I try to use it in my project (4.19) it tells me that the binaries are not compatible, I tried forcing a rebuild, but it don´t work, how can I solve this?

When I try to use it in my project (4.19) it tells me that the binaries are not compatible, I tried forcing a rebuild, but it don´t work, how can I solve this?

I'm on 4.19.2 and when it asked me to rebuild it worked @juangea I know that's not a solution, but hopefully it is helpful in some way.


noCodeFrame respects noStackTrace

Summary
When --noStackTrace flag is set, code frames should not be printed as mentioned here
Test plan
New snapshot test cases added.

@ychi ah no sorry, these use ansi. I take it all back, please revert last commit 😅

@SimenB, thank you for helping to revert. There seems to be an CI error for windows before adding wrap?

@ychi seems transient to me, let's see what CI says now 🙂

Codecov Report

Merging #9866 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #9866   +/-   ##
=======================================
  Coverage   64.39%   64.39%
=======================================
  Files         290      290
  Lines       12361    12361
  Branches     3056     3054    -2
=======================================
  Hits         7960     7960
  Misses       3761     3761
  Partials      640      640

Impacted Files
Coverage Δ

packages/jest-message-util/src/index.ts
73.82% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 32aaff8...69c74e1. Read the comment docs.


YouTube demands me to "sign in to confirm my age" -> how do i have yt-dlp tell my age to YouTube?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
how do i evade this
ERROR: [youtube] mJgC9LEBTuQ: Sign in to confirm your age. This video may be inappropriate for some users.

Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Two options:

Pass cookies from your browser where you're logged into an account that is either age-verified or is located a region that doesn't require age-verification

Try this plugin


remove k/v generics from the trie

This PR removes much incidental complexity arising from making K and V arguments to trie methods generic, when in reality we rarely use this generality, and moreover only in unit tests.
In addition, we now make use of DataAccessLayer, which uses ScratchGlobalState, which wraps LmdbGlobalState. This is an intermediate refactoring step to reduce the unnecessary complexity in the trie implementation.
Going forward, we can build on this work by collapsing LmdbGlobalState and ScratchGlobalState together into a cache-aware type.
This PR also integrates cache/scratch semantics into exec/commit.
Execute + commit is now replaced by exec + apply (commits to cache), + commit_to_disk.
This PR also refactors WasmTestBuilder to remove it's generality related to tries and EngineState, and also integrates ScratchGlobalState more closely.

#3327 - it's going to need to be rebased on latest from dev if we keep this around

This PR

I don't understand.
This here is an issue, not a PR.


How to avoid the selector taking up entire width?

With the default styling, the component stretches and takes up the entire width of the display. How can I force it to take up only the space that is required (e.g. according to the length of the text of the buttons inside)?
Currently I am using style={{width: '50%'}} as prop but it is not a elegant solution since it will vary according to display sizes.

You can wrap it in a View component and adjust the View component size accordingly


[fix](partial update) compaction may cause update failue

Proposed changes
Issue Number: close #xxx
Under normal circumstances, get_rowset_by_ids does not need to consider the stale rowset, in other word, if a rowset id is not found in the normal rowset, we can ignore it. This is because even if we handle stale rowset here, we need to recalculate the new rowset generated by the corresponding compaction in the publish phase.
However, for partial update, ignoring the stale rowset may cause some keys to not be found in the flush phase (lookup_row_key returns KEY_NOT_FOUND), and thus be mistaken as new keys in the flush phase, which will cause the load to fail in the following two cases:

when strict_mode is enabled, new keys are not allowed to be added.
Some columns that need to be filled are neither nullable nor have a default value, in which case the value of the field cannot be filled as a new key, leading to a failure of the load.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


request: analytics and song recording

At the moment, blockify is able to detect the song which is being played. I think it would be great if blockify could store the songs that are being played and figure out a title from the metadata it receives.
I propose a button "record" which stores the next and subsequent songs as separate mp3s or flacs.
Ideally, blockify would also create a database, where the titles of the songs are stored and the number of times you played them.
Combining the information, if a commercial comes by you just play at random one of the songs stored . You could use the database to compute a probability or pick a song from a similar genre.
Storing is likely legal if you have a paid license.  If you don't have a paid license, the situation is uncertain, see https://en.wikipedia.org/wiki/Radio_music_ripping#Legal_issues, but you could add a disclaimer that the user should only use the button if he has a paid license or be prepared to pay for possible damages.

I'm sorry but ripping music is not something i want to add to blockify.
If you want to play any kind of music during ads you can use the interlude player.
Adding a weighting mechanism to the interlude player might be an interesting feature for the future though.


⚠ ClamAV has degraded performance

In 16f9c47, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 77 ms

Resolved: ClamAV performance has improved in e747513 after 5 minutes.


Feature Request: Add Mixins

Stimulus recommends using the mixin pattern to add shared functionality to controllers, eg
import { useIsMobile } from "./mixins/useIsMobile";
export default class extends MainController {
  connect() {
    useIsMobile(this);
  }
  message() {
    if(this.isMobile()) {
      console.log("This is mobile");
    }
  }
}

export const useIsMobile = (controller) => {
  Object.assign(controller, {
    isMobile() {
      return window.innerWidth < 991;
    },
  });
};

Unfortunately the function isn't resolved in PhpStorm

could please share a sample project?

There are plenty of examples in the stimulus-use project, eg https://github.com/stimulus-use/stimulus-use/blob/main/playground/controllers/dropdown_controller.js

Got it, thank you. So far I am not sure that it can be fixed properly, but looks like I can "relax" resolve for this expression. I will try to take a look this week.


You might have unsuporrted firmware version None

Describe the bug
After the upgrade to HA Core 2022.2.8 unsupported firmware error
Version

HA version? 2022.2.8
HA Bosch component version? 0.17.3

** Debug SCAN **
** IMPORTANT **
Go to Developer tools in Home Assistant, choose Service tab and choose bosch.debug_scan
Download file to your computer and upload it somewhere eg. https://jsonblob.com/
bosch_scan.zip

Restarting HA fix the issue


[New Feature] Add import and export coffeecard

[Add import and export coffeecard]
prerequisite
Export needs ID figured out
description
Import coffee card (json file)
Export button that can export json.
Insert the steps to complete to fully implement this feature

[x] import
[x] obtain respective element in local storage
[x] export

Resources
https://developer.mozilla.org/en-US/docs/Web/API/File_API/Using_files_from_web_applications

Fixed by #71


Adding an attribute to existing product type

Describe the bug
After the initial creation of the product type, there might be a requirement to add a new optional attribute to it(which will later be used to update the products accordingly). Currently the API does not seem to allow such a change.
To Reproduce
Consider the following integration test in groovy :
def productTypeRequest = ProductTypeImportRequestBuilder.of()
                            .resources([ProductTypeImportBuilder.of()
                                                .key("default")
                                                .name("default")
                                                .description("another field")
                                                .attributes([AttributeDefinitionBuilder.of()
                                                                     .name("color")
                                                                     .label(CommerceToolsProductWriter.createLocalizedString("de", "Farbe"))
                                                                     .isRequired(false)
                                                                     .type(AttributeSetTypeBuilder.of()
                                                                             .elementType(AttributeTextType.of())
                                                                             .build())
                                                                     .build(),
                                                             AttributeDefinitionBuilder.of()
                                                                     .name("size")
                                                                     .label(CommerceToolsProductWriter.createLocalizedString("de", "Größe"))
                                                                     .isRequired(false)
                                                                     .type(AttributeSetTypeBuilder.of()
                                                                             .elementType(AttributeTextType.of())
                                                                             .build())
                                                                     .build()
                                                ])
                                                .build()]).build()

def response1 = client.withProjectKeyValue("...").productTypes()
                    .importSinkKeyWithImportSinkKeyValue("producttypes")
            .post(productTypeRequest)
            .executeBlocking()
            .getBody()
// wait for a while
def response2 = client.withProjectKeyValue("...").productTypes()
                    .importSinkKeyWithImportSinkKeyValue("producttypes")
            .importOperations()
            .get().executeBlocking().getBody()

at first, both the response1 and response2 will be completed successfully, afterwards if we do it again, but with an extra attribute weight :
def productTypeRequest = ProductTypeImportRequestBuilder.of()
                            .resources([ProductTypeImportBuilder.of()
                                                .key("default")
                                                .name("default")
                                                .description("another field")
                                                .attributes([AttributeDefinitionBuilder.of()
                                                                     .name("color")
                                                                     .label(CommerceToolsProductWriter.createLocalizedString("de", "Farbe"))
                                                                     .isRequired(false)
                                                                     .type(AttributeSetTypeBuilder.of()
                                                                             .elementType(AttributeTextType.of())
                                                                             .build())
                                                                     .build(),
                                                             AttributeDefinitionBuilder.of()
                                                                     .name("size")
                                                                     .label(CommerceToolsProductWriter.createLocalizedString("de", "Größe"))
                                                                     .isRequired(false)
                                                                     .type(AttributeSetTypeBuilder.of()
                                                                             .elementType(AttributeTextType.of())
                                                                             .build())
                                                                     .build(),
                                                             AttributeDefinitionBuilder.of()
                                                                     .name("weight")
                                                                     .label(CommerceToolsProductWriter.createLocalizedString("de", "Gewicht"))
                                                                     .isRequired(false)
                                                                     .type(AttributeSetTypeBuilder.of()
                                                                             .elementType(AttributeTextType.of())
                                                                             .build())
                                                                     .build()
                                                ])
                                                .build()]).build()

def response1 = client.withProjectKeyValue("...").productTypes()
                    .importSinkKeyWithImportSinkKeyValue("producttypes")
            .post(productTypeRequest)
            .executeBlocking()
            .getBody()
// wait for a while
def response2 = client.withProjectKeyValue("...").productTypes()
                    .importSinkKeyWithImportSinkKeyValue("producttypes")
            .importOperations()
            .get().executeBlocking().getBody()

Then reponse2 will contain the following error : The given attributes must be equal to the type attributes.
Expected behavior
The extra attribute definition should be imported without errors, or there has to be another way to introduce a new attribute to the product-types/products.

Thanks alot for reporting this issue. We can reproduce it and are currently investigating it further. we keep you posted.

This issue is fixed and deployed now.

Hi @katmatt , Thanks a lot for taking time to look at this issue. The behavior has changed now after the deployment, response2 in the above example now reports that the attributes are successfully imported, however the actual product type(in merchant center) does not reflect the said changes, and if I try to create a new variant with the additional attribute, I will receive an error that the attribute does not exist in the given type. I appreciate if you again take a look at the example above and let me know if my approach is correct.

Hi @katmatt , any update on this issue? thanks!

Hi @dpasdar ,
Sorry for not keeping you posted, but today we deployed a fix for this issue.
Please let us know if it works for you!

Thanks a lot, I can confirm that the issue is fixed now


Move acorn-jsx to devDependencies.

Correct me if I am wrong, but it does not seem to be used outside of tests.

@wtgtybhertgeghgtwtg can you merge latest into this branch? travis should pass now!

Coverage remained the same at 98.788% when pulling fb65030e5a4afdd8f0d1b39f12b1e2b06f9cc4f8 on wtgtybhertgeghgtwtg:master into e327eb8c51558b46a4cf2c8cc0326590b9ebf21b on evcohen:master.

Rebased.


Background colour

Is there a way to set the background colour of the area around the wizard page and title?
I need to set the whole background to the control colour, ie not white in the wizard control area. Setting .BackColor for all the controls in the WizardControl affects only wizard page area and the title text area - the surrounding panels stay white.
Originally posted: 2014-03-14T07:50:32

To set the background to SystemColors.Control, set the ClassicStyle property on WizardControl to BasicStyle.
Originally posted: 2014-03-17T07:19:06


Docs

Improved the Docs and added a Docusaurus package for the High Level Doc stuff

Closing this in favor of #66


refactor: Move all bindings code to bindings path

Same comment as #2117

I'm sure something will break somewhere, but I'm really not sure what, and there's only one way to find out...

I'll be on tomorrow to fix anything that does break


NixOS 17.09: update container example in NixOS manual

Issue description
Hi guys,
After reading the NixOS manual section on imperative container management, I followed the instructions there to run a web server in the container but then wasn't able to access the web server from the host using the curl command specified in the example. It looks like the example should also mention to allow incoming traffic on port 80 in the container config?
Steps to reproduce
Run all the commands specified in the NixOS manual section on imperative container management up to the point where you update the container's config to run a web server:
# nixos-container update foo --config 'services.httpd.enable = true; \
       services.httpd.adminAddr = "foo@example.org";'

Trying to run the next line in the example
# curl http://$(nixos-container show-ip foo)/

will result in curl hanging as packets are dropped, which you can see by running e.g. tcpdump. Adding this line to the container's config:
networking.firewall.allowedTCPPorts = [ 80 ];

fixed it for me. Maybe this should be mentioned in the manual? By the same token, it might also be worth mentioning that if Network Manager is running, you'll have to add this line for the example to work:
networking.networkmanager.unmanaged = [ "interface-name:ve-*" ];

I know this is mentioned in a later section, but I reckon it's best to make it explicit? In fact, even though Network Manager is disabled by default, some modules, like Gnome 3, enable it automatically so whoever reads the example may not be aware Network Manager is up just because they didn't explicitly enable it in their configuration.
Technical details

System: NixOS: 17.03.882.d1c477e007 (Gorilla)
Nix version: 1.11.8
Nixpkgs version: 17.03.882.d1c477e007

I guess the example was written in times when firewall was off by default.
Created a PR for that (https://github.com/NixOS/nixpkgs/pull/25180)

but sorry about your second issue - I have no opinion on that. Unfortunately I've marked my PR as "Closes #25174", so this issue may be autoclosed without resolution on this.

cool, no prob. i only reported the issue cos they asked me to on #nixos. but yah, even if in general docs can be improved, there's enough info that peeps can easily figure out what's missing or ask on #nixos just like I did. I guess that was a long way to say: don't stress about it :-)


Class League\Fractal\Manager does not exist

Hello, I get this error when try to upload to my vps and run composer, but in local is working. any idea why? thanks

4 month ago, i have solved , but now i dunno why still get message and didnt solve
{
  "code": 500,
  "errors": {
    "message": "Target class [League\\Fractal\\Manager] does not exist."
  }
}

i have remove vendor folder, composer self-update && composer dump-autoload then composer install

Did you find an answer to this @yogithesymbian ? The same happened to me when doing composer update and updating from 0.17 to 0.2.1. Looking in the src folder the files are simply missing, does not make any sense...


Default Sources Do Not Work after Fork

Checklist:

[x] I updated to the latest version available
[x] I cleared the cache of my browser

Release with the issue:
1.3.1
Last working release (if known):
NA
Browser and Operating System:
NA

Description of problem:
Preset custom sources do not work, likely broken when repo was forked to use the Android TV remote instead of a Samsung TV.

Default sources should launch app.
Javascript errors shown in the web inspector (if applicable):
NA

Additional information:
It should be possible using either the Android TV media_player or remote entities as described here. Prefer the latter so the dependency on media_player_id can be removed.

Slider card still relies on media_player so can't remove it. Found this great guide on deep linking which I'll pull from when updating sources.
https://community.home-assistant.io/t/android-tv-remote-app-links-deep-linking-guide/567921

Changing source now fixed in v.1.3.2
NOTE: I was not able to find the correct source/activity names for a couple of the entries I included in the new expanded default sources list, and did not test many of them as I did not have accounts for those services. If you find that any of the default sources do not work but do know the correct source/activity name, please let me know so that I can correct it.


Allow setting all arguments for chat.postMessage

Small PR allowing the use of extra options for chat.postMessage, so bot name and other parameters can be changed too.

Thanks for this! :+1:


Install requires urllib3 without explicitly specifying a version

The sdk requires urllib3 as a dependency in setup.py line 24, but doesn't explicitly specify a version number. This causes a dependency conflict in projects that require a different version of urllib3 as the sentry-sdk will always try to install the latest version of urllib3.

Which tool shows you this dependency conflict? I'd argue it's broken for assuming no version range means always latest version.

Which tool shows you this dependency conflict? I'd argue it's broken for assuming no version range means always latest version.

I'm using a number of libraries, google-cloud-storage, nameko, requests among some others. They all specify the version range of urllib3 as >=1.21.1,<1.25. However, urllib3 recently (yesterday) released a new version hence the conflict

We don't require the latest version of urllib3. It even shows required: Any in your screenshot. I don't understand where the conflict is, or more importantly, what the actual problem is that results out of this for you.
My theory is that you have multiple pip install commands, some of which run with -U, that overwrite parts of the dependency tree that the previous commands have built. But that would not be the problem of the Sentry SDK, or even something we could fix or work around in any way.

Without pinning a specific version of urllib, sentry will always install the latest version (which is not what other libraries might need). To avoid conflicts with other libraries that also require urllib (could be a different version), I have to list sentry as the first item in my requirements.txt just to make sure any other dependency that comes after sentry uninstalls the version already installed by sentry and replaces it with the specific version that it requires. Wouldn't it be better to pin a specific version so as to keep producing reproducible builds?

Wouldn't it be better to pin a specific version so as to come up with reproducible builds?

I don't see how reproducible builds or anything close to it solve your problem. You're asking me to pin urllib3 in sentry-sdk to the version your other dependencies use just so there is no conflict specifically in your application. This will introduce more conflicts for people whose dependency tree looks different.
The fact that pip install does not deal with version conflicts is not something this library can solve. You have to work around it somehow else.

Having the same problem, with two different libraries -- botocore and requests.  I now get these errors from doing a pip install if I don't explicit fix a version of urllib3:
ERROR: botocore 1.12.145 has requirement urllib3<1.25,>=1.20; python_version >= "3.4", but you'll have urllib3 1.25.2 which is incompatible.
ERROR: requests 2.21.0 has requirement urllib3<1.25,>=1.21.1, but you'll have urllib3 1.25.2 which is incompatible.

It doesn't seem that sentry is doing anything wrong per se, but it isn't playing by the same rules as these other libraries.

I really have no idea how to even work around this in the sdk without breaking the dependency trees of even more people. Install a specific version of urllib3 explicitly to solve this problem or use something with reasonable dependency resolution, like poetry.


ISPN-16279 Non blocking methods for Query API

Add non blocking methods that can be used easily by the RemoteCache Query API
https://issues.redhat.com/browse/ISPN-16279

[x] JavaDoc for the new methods.

Hi @pruivo. Thanks for contribution. Let's wait the CI. When I came back from the PTO I'll have a look at it to evaluate the implications.

The error on RemoteStoreTest seems to be not related to this contribution. We don't use queries there. Running it locally it works. So I'm merging it

Merged, thanks

thanks @fax4ever


Emem & Comp variable

Is there any way we could get Emem & Comp naming variables? These values are constantly changing and it would be nice if companion would follow along.

I don't currently see a way of gathering this info via the API. It's possible it's been added since that documentation was last updated, but I dug a little bit and didn't see it. Might be worth reaching out to NewTek and seeing if they can add it to their API

I’ll reach out to our technical rep and see what he can gather for us. I’ll report back on what he says regarding that API call

Currently not supported with API. will keep an eye on updates and open a ticket if support is added in the future.


Add Feedback Form

Description 📜
Add a feedback template
Fixes #130

This would be great if you provide any of this   📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Single page website

[x] Others

Checklist ✅
]

[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm Hacktoberfest contributor

Any tutorials or documentation have you added
(made by you only not from google)

[ ] Tutorial
[ ] Documentation

Screenshots / Gif (Compulsory) 📸

looks good @amelia2802  can you pls add one small clip showing its responsiveness

Here it is:
!Preview

@amelia2802  Thnx for Contributing !! Hope you learned something new , Don't forget to star repo and follow me for further such amazing repos 👍


Data values on enumerated elements don't update with collection changes

If a collection is enumerated with rv-each, then data is set on each element via rv-data-*, removing elements from the lists messes up the associated data retrieved by $.data.
This JSFiddle illustrates a direct example. Click the Log data and Pop user buttons to see how the data set doesn't jive with the popping.
It's not clear to me what would be causing the issue here. Can anyone advise if this is likely a problem with the rivets code or a greater issue?

I think I fixed it for you, can you confirm that this is the behavior you wanted?
https://jsfiddle.net/Vornagreg/64fx59gt/
The pertinent bits:
Updated you call to bind:
rivets.bind(document.getElementById('details'), {
    model : {
        users: users,
        killUser: killUser,
        logData: logData}
    });

Then access it in the DOM via model.users and model.killUser etc.
This issue crops up all the time. You will always want to wrap most everything you bind in an object.
I cannot explain exactly why, but it happens all the time and almost always the solution is to wrap it in an object.
@Leeds-eBooks Might be able to explain it better.
Please close if this helps you. I don't believe this is a bug as we have been very consistent in saying that people should always wrap it.

Thanks for proposing a solution. I'm not seeing the problem resolved in that example.
The issue is that information stored in $.data becomes inconsistent as the list is modified. In the fiddle you posted I'm still seeing that, after popping the first user off, $('p[data-name="Second"]').data() gives me an object with name: "First", whereas one would expect it to yield name: "Second".
Do you see the issue I'm referring to?

@mhanders use built-in browser DOM inspector to debug this.
$('p[data-name="Second"]').data() seems to be wrong because of caching stuff. See answer here: http://stackoverflow.com/questions/21588044/jquerys-data-attribute-does-not-update-when-element-data-changes

Yeah, Thanks for clarifying,  I see what you are saying now.
Isnt that a problem with jquery though?
Cause if you look at the DOM, the attribute is in fact updated:

Considering it is def a jQuery issue and not a rivets issue. As illustrated by @Namek
@mhanders  Can you close this?

Definitely. Thanks for pinning down!

This is a know rivets issue.
In the each binder, when the collection is already rendered, it reuses the elements. See https://github.com/mikeric/rivets/blob/master/src/binders.coffee#L193
It uses pop method of iterated views and then remove its element (the last one). When shift or slice method is used in the observed array, the elements <> data order desync.
See this updated fiddle to demonstrate that theres no problem when using pop. Also instrumented to track the elements order


Fix putorpatch not running with the static validator

Which issue this PR addresses:
rather than make #1803 bigger, I split it out
What this PR does / why we need it:

enables static validation for the putorpatch tests
fixes the putorpatch tests to reasonably pass
cleans up some test infra too

Test plan for issue:
is tests
Is there any documentation that needs to be updated for this PR?
N/A

old


Add changelog for loadouts

About the PR

SS14.Changelog did not support commas when this pr was merged, causing the bot to not make a changelog. A fix was made for ss14.changelog for this. But I will manually make a commit here
Ama let yaml linter finish just in case

Check https://github.com/space-wizards/SS14.Changelog/pull/13


Order detail discussion

This PR completes the Discussion tab on the Order Detail overlay.
It should be test and merged in conjunction with this server PR.

@morebrownies One note on this one... we've decided to punt on the message read indicator for the discussion tab, since there are complications with this being a group chat. Basically, the server would have to be refactored to keep track of which chat participant read the message. Also, the design would need to be refactored as well... for example, let's say you're the buyer and the message has the little green border... does that mean it was read by the moderator, vendor or both.
Anyways, I'm not sure such an indicator works on a group chat.

Basically, the server would have to be refactored to keep track of which chat participant read the message.

ah ok :(

Also, the design would possibly need some adjustment as well... for example, let's say you're the buyer and the message has the little green border... does that mean it was read by the moderator, vendor or both.

FWIW the unread indicators were intended to display which messages were previously unread by you. So in your example above, if you're the buyer, and the vendor already read the most recent message, you (the buyer) would still see the green indicator because they haven't read it yet.


Can Get-RpcAlpcServer retrieve information from process inside a container?

Does Get-RpcAlpcServer can get the RpcAlpcServer from a process inside a container?
I had a container running with process CExecSvc.exe (PID 4880) but when I tried to get it with the above function it didn't find it:
PS C:\Windows\system32> Get-RpcAlpcServer| where {$_.ProcessName -eq "cexecsvc.exe"}
PS C:\Windows\system32> Get-RpcAlpcServer -ProcessId 4880
PS C:\Windows\system32>

I noticed that RPCView also can't get this information.
This is not critical for me because I used the workaround with Get-RpcServer on the file itself, just wondering if this is a bug or currently not possible.

Reference:
https://github.com/googleprojectzero/sandbox-attacksurface-analysis-tools/blob/c02ed8ba04324e54a0a188ab9877ee6aa372dfac/NtApiDotNet/Win32/RpcAlpcServer.cs#L91-L112

It might work if I expand the check to find \RPC Control\ anywhere in the name. This also affects AppContainers as well so might be worth doing. You could try and do it manually by extracting the path to the ALPC Port and then using the Get-RpcEndpoint command instead. e.g.
$name = (Get-NtHandle -ProcessId <PID OF EXE> | ? ObjectType -eq 'ALPC Port' | ? Name -Match 'RPC Control').Name
Get-RpcEndpoint -AlpcPort $name

I've not tested that though.

Yes, it does the job, thank you :)
PS C:\Users\Administrator> $name = (Get-NtHandle -ProcessId 4880 | ? ObjectType -eq 'ALPC Port' | ? Name -Match 'RPC Control').Name
PS C:\Users\Administrator> Get-RpcEndpoint -AlpcPort $name

UUID                                 Version Protocol Endpoint                        Annotation
----                                 ------- -------- --------                        ----------
75ef42c7-22f4-44a0-8200-9351cd316e01 1.0     ncalrpc  \Silos\532\RPC Control\cexecsvc


custom logger

hi, i implement my own ILogger, and could you tell me how can i set it to dragonboat ?
the default logs in dragonboat are written to stderr/stdout, so i want to use my own log system.
thanks.

Please use the factory function linked below.
https://github.com/lni/dragonboat/blob/7db91bd26f838034603319ae0a3ea50e49cb0126/logger/logger.go#L59


Fix axom klee tests and revert bp plugin mat nos

Description
Fixes the axom klee tests.
Basically reverts #19307

Type of change

[x] Bug fix~~
[ ] New feature
[ ] Documentation update
[ ] Other

How Has This Been Tested?

Built and ran on rzhound toss4. All tests pass.
Reminders:

Please follow the style guidelines of this project.
Please perform a self-review of your code before submitting a PR and asking others to review it.
Please assign reviewers (see VisIt's PR procedures for more information).

Checklist:

[x] I have commented my code where applicable.~~
[x] I have updated the release notes.~~
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] I have confirmed new and existing unit tests pass locally with my changes.~~
[ ] I have added new baselines for any new tests to the repo.
[ ] I have NOT made any changes to protocol or public interfaces in an RC branch.

merge w/ develop in #19393


Galaxy in AnVIL guide

Tasks:

[ ] Install tools as your own admin
[ ] Import/export data from workspace [export doesn’t work]
[ ] Import/export history from workspace
[ ] Import workflow (via url with /json or via .ga)
[ ] Export workflow (save .ga or save .json)
[ ] GTN: Run this tutorial on AnVIL
[ ] GTN: Add galaxy-on-anvil tag to tutorials

See Also: GCC 2023 Cofest notes

How about running particular analyses on Galaxy AnVIL like RNA-seq, machine learning, etc.?


build test failure: unable to find yaml config while it exists

[test]              /src:/src/..
[test]              warning: "./trueconsensus/dapps/..." matched no packages
[test]              warning: "./trueconsensus/db/..." matched no packages
[test]              warning: "./trueconsensus/evm/..." matched no packages
[test]              2018/08/01 01:35:30 Unable to read config file. Error:open /etc/truechain/tunables_bft.yaml: no such file or directory
[test]              --- FAIL: TestConfig (0.00s)
[test]              	assertions.go:256:
[test]              			Error Trace:	config_test.go:32
[test]              			Error:      	Expected nil, but got: &os.PathError{Op:"open", Path:"/etc/truechain/tunables_bft.yaml", Err:0x2}
[test]              			Test:       	TestConfig
[test]              	assertions.go:256:
[test]              			Error Trace:	config_test.go:33
[test]              			Error:      	Should not be: 0
[test]              			Test:       	TestConfig
[test]              FAIL
[test]              FAIL	trueconsensus/fastchain	0.024s
[test]              ?   	trueconsensus/fastchain/proto	[no test files]
[test]              ?   	trueconsensus/minerva	[no test files]
[test]              ?   	trueconsensus/client	[no test files]
[test]              ?   	trueconsensus/common	[no test files]
[test]              ?   	trueconsensus/logging	[no test files]
[test]              ?   	trueconsensus/snailchain	[no test files]

while expected output should be: PASS
I've tested this locally, abstracting the logic away from the codebase and it works.
So there's something that has not caught my attention yet.

This is not a issue.
If you had worked with docker image before, the  root path is the exported path.
in test.sh, please check:
export GOPATH=pwd:pwd/..


Support torch==1.9.1

It's currently unsupported

Should be fixed in #228 once we fix tests and rerun the scraper.

I think this is a dupe of #204 actually.


Load maps from bundled zip

If a .zip file is in the Maps directory and it contains the mapInfo.json file, Mapify should be able to decompress it in the Maps folder and move the original .zip to the system trash.
This will simplify manual map installation and reduce the support needed from people incorrectly unpacking maps.

With the new BepInEx distribution system, this isn't required anymore.


App Service Migration Assistant Tool failed at Publish site content Step

I am facing a migration failed issue over last step i.e Publish site content on Azure App Service Migration Tool, although App Service is created with empty content. Here below is screenshot of the error that I am facing.

Hi @Asrar0786,
I am wondering if this may be due to basic auth not being enabled - it appears the site was successfully created but the final content publish step failed. If it is due to basic auth not being enabled, this won't work in the GUI, however you can do the final content upload manually using steps like outlined in this issue (https://github.com/Azure/App-Service-Migration-Assistant/issues/31#issuecomment-620884855) - to either find an existing or create the site content .zip and then drag-and-drop that .zip content to the wwwroot location using the .scm site.


Allow changing GitHub server hostname (for self-hosted/enterprise instances)

Why do we need this ?
I would like to use this tool with an internally hosted GitHub instance.
Acceptance Criteria
I would be OK with this being something that can be set via env-var or config file.
It does not need to be something that supports live-reloading or even something that's likely to be changed often.
Further Comments / References
Nothing immediately comes to mind; it should just be a hostname change. Nothing about the API changes with self-hosted.

Hey @kquinsland, we've not gotten around to this yet, but perhaps you could create a fork and replace references to github.com with your own host. As long as the API contracts are identical, I feel it'll work out of the box. :)
If it does work for you, let me know and we'll add it to the README or Wiki. :)

If it does work for you, let me know and we'll add it to the README or Wiki. :)

I tried this and found I had to make quite a few small patches here and there before I ran out of time. I don't recall precisely where I got stuck (it's been 6 weeks...) but there was some issue(s) I had w/r/t auth that I wasn't able to fully grock.

Had a spare sec to go digging.
# So you can see the commit to compare my changes against
❯ git rev-parse HEAD
8605f85a1053d94a49aea9e63d34de153cca35d8
# Generate a diff of all the things I had to change
❯ git diff --name-only | while read -r file; do git diff "$file"; echo ""; done > patches-i-made

And here's patches-i-made:
❯ cat patches-i-made
diff --git a/Dockerfile.dev b/Dockerfile.dev
index 1c747f8..e5c4922 100644
--- a/Dockerfile.dev
+++ b/Dockerfile.dev
@@ -37,6 +37,9 @@ RUN /opt/venv/bin/pip install --upgrade pip
 RUN --mount=type=cache,target=/root/.cache/pip /opt/venv/bin/pip install -r requirements.txt -r dev-requirements.txt

 WORKDIR /app
+# TODO: this should be intelligent; uname --machine or similar?
+RUN curl -fsSL -o /usr/local/bin/dbmate https://github.com/amacneil/dbmate/releases/download/v1.16.0/dbmate-linux-arm64
+RUN chmod +x /usr/local/bin/dbmate
 RUN mkdir -p /etc/cron.d && mv /app/setup_utils/cronjob.txt /etc/cron.d/cronjob
 RUN chmod +x /app/setup_utils/start.sh /app/setup_utils/init_db.sh /app/setup_utils/generate_config_ini.sh
 RUN mv /app/setup_utils/supervisord.conf /etc/supervisord.conf

diff --git a/Dockerfile.prebuilt b/Dockerfile.prebuilt
index e942b33..c22a962 100644
--- a/Dockerfile.prebuilt
+++ b/Dockerfile.prebuilt
@@ -13,7 +13,7 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
         curl \
     && curl -fsSL https://deb.nodesource.com/setup_22.x | bash - \
     && apt-get install -y nodejs \
-    && curl -fsSL -o /usr/local/bin/dbmate https://github.com/amacneil/dbmate/releases/download/v1.16.0/dbmate-linux-amd64 \
+    && curl -fsSL -o /usr/local/bin/dbmate https://github.com/amacneil/dbmate/releases/download/v1.16.0/dbmate-linux-arm64 \
     && chmod +x /usr/local/bin/dbmate \
     && apt-get clean \
     && rm -rf /var/lib/apt/lists/*

diff --git a/backend/analytics_server/mhq/exapi/github.py b/backend/analytics_server/mhq/exapi/github.py
index 50cd192..194df9c 100644
--- a/backend/analytics_server/mhq/exapi/github.py
+++ b/backend/analytics_server/mhq/exapi/github.py
@@ -25,7 +25,8 @@ class GithubApiService:
     def __init__(self, access_token: str):
         self._token = access_token
         self._g = Github(self._token, per_page=PAGE_SIZE)
-        self.base_url = "https://api.github.com"
+        # self.base_url = "https://api.github.com"
+        self.base_url = "https://github.corp-github-server.net/api/v3"
         self.headers = {"Authorization": f"Bearer {self._token}"}

     @contextlib.contextmanager
@@ -53,6 +54,7 @@ class GithubApiService:
         except GithubException as e:
             raise e

+        LOG.error(f"FIND ME. Orgs got: {orgs}")
         return orgs

     def get_repos(
@@ -89,6 +91,7 @@ class GithubApiService:
         try:
             return self._g.get_repo(f"{org_login}/{repo_name}")
         except GithubException as e:
+            LOG.info(f"Something Broke! e: {e}")
             raise e

     def get_repo_contributors(self, github_repo: GithubRepository) -> [Tuple[str, int]]:

diff --git a/backend/analytics_server/mhq/service/code/sync/etl_github_handler.py b/backend/analytics_server/mhq/service/code/sync/etl_github_handler.py
index 17f042c..6e6a2dc 100644
--- a/backend/analytics_server/mhq/service/code/sync/etl_github_handler.py
+++ b/backend/analytics_server/mhq/service/code/sync/etl_github_handler.py
@@ -71,11 +71,18 @@ class GithubETLHandler(CodeProviderETLHandler):
         :param org_repos: List of OrgRepo objects
         :returns: List of GitHub repos as OrgRepo objects
         """
+        LOG.info(f"Inside get_org_repos. org_repos: {org_repos}")
+
+        for org_repo in org_repos:
+            LOG.info(f"Inside get_org_repos. org_repo: {org_repo}")
+            LOG.info(f"Inside get_org_repos. org_name: {org_repo.org_name}")
+            LOG.info(f"Inside get_org_repos. name: {org_repo.name}")
         github_repos: List[GithubRepository] = [
             self._api.get_repo(org_repo.org_name, org_repo.name)
             for org_repo in org_repos
         ]
-
+        # Boom. Confirmed that we never make it this far!
+        LOG.info(f"Inside get_org_repos. github_repos: {github_repos}")
         repo_idempotency_key_org_repo_map = {
             org_repo.idempotency_key: org_repo for org_repo in org_repos
         }

diff --git a/backend/analytics_server/mhq/service/code/sync/etl_handler.py b/backend/analytics_server/mhq/service/code/sync/etl_handler.py
index 47dfa95..a15e8d8 100644
--- a/backend/analytics_server/mhq/service/code/sync/etl_handler.py
+++ b/backend/analytics_server/mhq/service/code/sync/etl_handler.py
@@ -47,8 +47,11 @@ class CodeETLHandler:
         if not self.etl_service.check_pat_validity():
             LOG.error("Invalid PAT for code provider")
             return
+        LOG.info(f"about to get org_repos....")
         org_repos: List[OrgRepo] = self._sync_org_repos(org_id, provider)
+        LOG.info(f"got the following org_repos: {org_repos}")
         for org_repo in org_repos:
+            LOG.info(f"Attempting work on: {org_repo}")
             try:
                 self._sync_repo_pull_requests_data(org_repo)
             except Exception as e:
@@ -58,12 +61,16 @@ class CodeETLHandler:
                 continue

     def _sync_org_repos(self, org_id: str, provider: CodeProvider) -> List[OrgRepo]:
+        LOG.info(f"about to try...")
         try:
             org_repos = self.code_repo_service.get_active_org_repos_for_provider(
                 org_id, provider
             )
+            LOG.info(f"step 1, org_repos: {org_repos}")
             org_repos = self.etl_service.get_org_repos(org_repos)
+            LOG.info(f"step 2, org_repos: {org_repos}")
             self.code_repo_service.update_org_repos(org_repos)
+            LOG.info(f"step 3, org_repos: {org_repos}")
             return org_repos
         except Exception as e:
             LOG.error(f"Error syncing org repos for org {org_id}: {str(e)}")
@@ -136,6 +143,7 @@ def sync_code_repos(org_id: str):
     etl_factory = CodeETLFactory(org_id)

     for provider in code_providers:
+        LOG.info(f"Iterating through: {provider}")
         try:
             code_etl_handler = CodeETLHandler(
                 CodeRepoService(),
@@ -144,6 +152,7 @@ def sync_code_repos(org_id: str):
                 get_bookmark_service(),
                 get_settings_service(),
             )
+            LOG.info(f"code_etl_handler instantiated! {code_etl_handler}")
             code_etl_handler.sync_org_repos(org_id, CodeProvider(provider))
             LOG.info(f"Synced org repos for provider {provider}")
         except Exception as e:

diff --git a/backend/analytics_server/mhq/service/sync_data.py b/backend/analytics_server/mhq/service/sync_data.py
index 35e9abd..bb7bbd4 100644
--- a/backend/analytics_server/mhq/service/sync_data.py
+++ b/backend/analytics_server/mhq/service/sync_data.py
@@ -16,6 +16,7 @@ def trigger_data_sync(org_id: str):
     LOG.info(f"Starting data sync for org {org_id}")
     for sync_func in sync_sequence:
         try:
+            LOG.info(f"Starting sync_func: {sync_func}...")
             sync_func(org_id)
             LOG.info(f"Data sync for {sync_func.__name__} completed successfully")
         except Exception as e:

diff --git a/database-docker/Dockerfile b/database-docker/Dockerfile
index c12e785..25c9589 100644
--- a/database-docker/Dockerfile
+++ b/database-docker/Dockerfile
@@ -1,7 +1,7 @@
-FROM debian:buster
+FROM debian:bookworm
 ENV DEBIAN_FRONTEND noninteractive
 RUN apt-get update
-RUN echo "deb http://apt.postgresql.org/pub/repos/apt/ buster-pgdg main" > /etc/apt/sources.list.d/pgdg.list
+RUN echo "deb http://apt.postgresql.org/pub/repos/apt/ bookworm-pgdg main" > /etc/apt/sources.list.d/pgdg.list
 RUN apt-get -y install gnupg
 RUN apt-get -y install curl
 RUN apt-get -y install wget
@@ -9,7 +9,8 @@ RUN curl -sSL https://www.postgresql.org/media/keys/ACCC4CF8.asc | apt-key add -
 RUN wget --quiet -O - https://www.postgresql.org/media/keys/ACCC4CF8.asc | apt-key add -
 RUN apt-get update
 RUN apt-get -y install postgresql-14
-RUN curl -fsSL -o /usr/local/bin/dbmate https://github.com/amacneil/dbmate/releases/download/v1.16.0/dbmate-linux-amd64
+# TODO: this should be intelligent; uname --machine or similar?
+RUN curl -fsSL -o /usr/local/bin/dbmate https://github.com/amacneil/dbmate/releases/download/v1.16.0/dbmate-linux-arm64
 RUN chmod +x /usr/local/bin/dbmate
 WORKDIR /
 RUN mkdir ./db

diff --git a/env.example b/env.example
index 99fabc5..f0044d9 100644
--- a/env.example
+++ b/env.example
@@ -1,4 +1,4 @@
-ENVIRONMENT=dev
+ENVIRONMENT=prod
 POSTGRES_DB_ENABLED=true
 DB_INIT_ENABLED=true
 REDIS_ENABLED=true

diff --git a/web-server/pages/api/internal/[org_id]/utils.ts b/web-server/pages/api/internal/[org_id]/utils.ts
index 3503c01..359e52a 100644
--- a/web-server/pages/api/internal/[org_id]/utils.ts
+++ b/web-server/pages/api/internal/[org_id]/utils.ts
@@ -6,7 +6,8 @@ import { Integration } from '@/constants/integrations';
 import { BaseRepo } from '@/types/resources';
 import { db } from '@/utils/db';

-const GITHUB_API_URL = 'https://api.github.com/graphql';
+// const GITHUB_API_URL = 'https://api.github.com/graphql';
+const GITHUB_API_URL = 'https://github.corp-github-server.net/api/graphql';

 type GithubRepo = {
   name: string;
@@ -53,7 +54,8 @@ export const searchGithubRepos = async (
 };

 const searchRepoWithURL = async (searchString: string) => {
-  const apiUrl = `https://api.github.com/repos/${searchString}`;
+  //const apiUrl = `https://api.github.com/repos/${searchString}`;
+  const apiUrl = `https://github.corp-github-server.net/api/v3/repos/${searchString}`;
   const response = await axios.get<GithubRepo>(apiUrl);
   const repo = response.data;
   return [

diff --git a/web-server/src/content/Dashboards/ConfigureGithubModalBody.tsx b/web-server/src/content/Dashboards/ConfigureGithubModalBody.tsx
index 3ed2ab9..06b087f 100644
--- a/web-server/src/content/Dashboards/ConfigureGithubModalBody.tsx
+++ b/web-server/src/content/Dashboards/ConfigureGithubModalBody.tsx
@@ -51,7 +51,7 @@ export const ConfigureGithubModalBody: FC<{
     depFn(isLoading.true);
     checkGitHubValidity(token.value)
       .then(async (isValid) => {
-        if (!isValid) throw new Error('Invalid token');
+        if (!isValid) throw new Error('Invalid token-TEST');
       })
       .then(async () => {
         try {

diff --git a/web-server/src/utils/auth-supplementary.ts b/web-server/src/utils/auth-supplementary.ts
index e5aec2c..7b9c329 100644
--- a/web-server/src/utils/auth-supplementary.ts
+++ b/web-server/src/utils/auth-supplementary.ts
@@ -28,7 +28,8 @@ export const INTEGRATION_CONFLICT_COLUMNS = ['org_id', 'name'];

 export const validateGithubToken = async (token: string) => {
   try {
-    const response = await axios.get('https://api.github.com/user/repos', {
+    const response = await axios.get('https://github.corp-github-server.net/api/v3/user/repos', {
+    //const response = await axios.get('https://api.github.com/user/repos', {
       headers: {
         // @ts-ignore
         Authorization: `token ${dec(token)}`

diff --git a/web-server/src/utils/auth.ts b/web-server/src/utils/auth.ts
index 4cc2d93..5d751ed 100644
--- a/web-server/src/utils/auth.ts
+++ b/web-server/src/utils/auth.ts
@@ -31,23 +31,35 @@ export async function checkGitHubValidity(
   good_stuff: string
 ): Promise<boolean> {
   try {
-    await axios.get('https://api.github.com/user', {
+    //const response = await axios.get('github.corp-github-server.net/api/v3/', {
+    // works, but not correct? await axios.get('https://github.corp-github-server.net/api/v3/user', {
+    await axios.get('https://github.corp-github-server.net/api/v3', {
+    //await axios.get('https://api.github.com/user', {
       headers: {
         Authorization: `token ${good_stuff}`
       }
     });
     return true;
   } catch (error) {
-    return false;
+    console.error(`Couldn't validate GH token. error: `, error);
+
+    //return false;
+    // For reasons that I don't understand, we still end up here.
+    // Despite the face that I can curl properly inside the container and get the
+    // results that I expect / show auth is ok...
+    return true;
   }
 }

 const PAT_SCOPES = ['read:org', 'read:user', 'repo', 'workflow'];
 export const getMissingPATScopes = async (pat: string) => {
   try {
-    const response = await axios.get('https://api.github.com', {
+    const response = await axios.get('https://github.corp-github-server.net/api/v3', {
+    //const response = await axios.get('https://api.github.com', {
       headers: {
-        Authorization: `token ${pat}`
+        //Authorization: `token ${pat}`
+        Authorization: `token ghp_zTRQW<...token here ...>`
+
       }
     });

@@ -57,7 +69,9 @@ export const getMissingPATScopes = async (pat: string) => {
     const userScopes = scopesString.split(',').map((scope) => scope.trim());
     return PAT_SCOPES.filter((scope) => !userScopes.includes(scope));
   } catch (error) {
-    throw new Error('Failed to get missing PAT scopes', error);
+    // i - again - do not know why I end up here...
+    return PAT_SCOPES.filter((scope) => !PAT_SCOPES.includes(scope));
+    //throw new Error('Failed to get missing PAT scopes', error);
   }
 };

The changes fall into two broad groups:

Changes required to get this booting inside a modern debian (bookworm should be used, buster is EOL) container and on an ARM based mac.
Changes required to get past steps that failed successfully / tracing code flow / trying to figure out why the app was telling me "no" but the token I was using ... appeared to work when doing a curl .... request manually.


top-level: fix nix-shell eval w/nixUnstable

Motivation for this change
For a full description of the underlying issue please read
https://github.com/NixOS/nix/issues/4003
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Note: IMHO we should backport this to 20.09 and 20.03 to allow users of nixUnstable to use nix-shell on their nixpkgs checkouts.

Backported to 20.03 as 732684b720f829056dcb62380c2c17e4d3ebd947, to 20.09 as c1dbaed781e769cb9d5383b031e5ef82e3acbeae.

I just upgraded to nix 2.4 stable and found this issue when trying to reproduce a nixpkgs 20.03 build
It is quite unfortunate that with nix >= 2.4, you cannot reproduce old builds at all now, because they don't evaluate, breaking the reproducibility story.
Is there something that could be done in nix to make that possible?

I'm trying to fix that over here but that isn't as straight forward.


Bug Report  azure_firewall duplication in variables

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform:
azure provider:
module:
Description
Describe the bug
as im looking i see 2 spots in variables.tf for  azure_firewall
lines 483 and 529
when i put true on both i see the one at 483 creates the Firewall with a private ip as and has the associated subnets  but 529 only creates the fw without any real association and in network watcher you can not even see it
can i understand the differences between the 2
sorry i have tried to review documents ad i cant seem to find anything
Steps to Reproduce

step 1
step 2
you get it...

Screenshots

Additional context

Hi @frankscalzo,
Are you consuming the module from the registry, rather than cloning it and trying to use it?
If so, please review these Wiki docs on how to use the connectivity module, which is where the firewall is deployed/managed from:

https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/wiki/[Examples]-Deploy-Connectivity-Resources
https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/wiki/[Examples]-Deploy-Connectivity-Resources-With-Custom-Settings
https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/wiki/[Variables]-configure_connectivity_resources

Let us know if these help or not and if you need anything further
Thanks
Jack


Use assertEqual instead of assertEquals for Python 3.11 compatibility.

The deprecated aliases have been removed in python/cpython#28268

already done in another PR


Cleaned up service bus module & added output descriptions

Change

Cleaned up service bus module

Convert json(null) into null
Removed excess brackets

added output descriptions

Type of Change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update (Wiki)

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] My corresponding pipelines / checks run clean and green without any errors or warnings
[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (readme)
[x] I did format my code

This PR is progress towards closing #410 and #238, but doesn't alone close it. So I am unlinking issues.


ER Diagram model

Please upload the ER Diagram model based on the Data requirements (DataRequirement.md)

This task is done. Now, it is ok to move on setting up the application


Skipping the kubernetes-csi/external-snapshotter images to update

Issue #, if available:
Description of changes:
Skipping to update the images. for mpvarma/update-checking-csi-snapshotter as this image is bumped but not published to the external repo.
Error updating image digests: ImageNotFoundException: The image with imageId {imageDigest:'null', imageTag:'v8.2.0-eks-1-29-30'} does not exist within the repository with name 'kubernetes-csi/external-snapshotter/snapshot-validation-webhook' in the registry with id '316434458148'

https://prow.eks.amazonaws.com/view/s3/prowdataclusterstack-316434458-prowbucket7c73355c-1n9f9v93wpjcm/logs/dev-release-1-29-postsubmit/1869869112969138176#2:build-container-build-log.txt%3A8080
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/approved
/unhold

/approved

/lgtm

/approved


Build(deps-dev): Bump @testing-library/jest-dom from 5.16.0 to 5.16.1 (#2120)

Pull request opened by the merge tool on behalf of #2120

Merged in 7ae6969a1b3608c21dcfb487bdd0e0aaff805903


Fix extract_dir for oracle-(jre|jre-server) 8u141-b15

Seems that auto-pr does not work with match groups. Should I create an issue?

Strange, auto-pr.ps1 simply calls checkver.ps1. If I run it locally it works. Maybe something strange with Powershell 6 on Linux again :grin:


Write a valid JSON

Pull request #40 introduced a bug where the out.json file is not a valid json.

Pull Request Test Coverage Report for Build 124

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 123:
0.0%

Covered Lines:
49

Relevant Lines:
49

💛  - Coveralls

I thought it would be covered by the tests but it is not I'll try to add a test that validates out.json file.

@bcoe This PR should merge before you push a new release. Or at least PR #40 should be reverted since it introduces a major regression.